summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* updated LICENSE yearv2.8.16origin/2.8Fabien Potencier2017-01-021-1/+1
|
* Merge branch '2.7' into 2.8Fabien Potencier2017-01-024-4/+4
|\ | | | | | | | | | | * 2.7: updated LICENSE year Fixed `@return self` with `$this`
| * updated LICENSE yearv2.7.23origin/2.7Fabien Potencier2017-01-025-5/+5
| |
| * fixed @return when returning this or staticFabien Potencier2016-12-273-7/+7
| |
* | fixed obsolete getMock() usageFabien Potencier2016-12-1911-71/+71
| |
* | Merge branch '2.7' into 2.8Fabien Potencier2016-12-1952-434/+434
|\ \ | |/ | | | | | | | | * 2.7: fixed obsolete getMock() usage [WebProfilerBundle] Display multiple HTTP headers in WDT
| * fixed obsolete getMock() usageFabien Potencier2016-12-1966-488/+488
| |
* | [Security] Fix testNicolas Grekas2016-12-171-1/+1
| |
* | Merge branch '2.7' into 2.8Fabien Potencier2016-12-142-1/+72
|\ \ | |/ | | | | | | | | | | | | * 2.7: [Validator] add Indonesian translation fixed CS [config] Fix issue when key removed and left value only [Security] AbstractVoter method supportsAttribute gives false positive if attribute is zero (0)
| * bug #20734 [Security] AbstractVoter->supportsAttribute gives false positive ↵Fabien Potencier2016-12-142-1/+72
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if attribute is zero (0) (martynas-foodpanda) This PR was merged into the 2.7 branch. Discussion ---------- [Security] AbstractVoter->supportsAttribute gives false positive if attribute is zero (0) | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Issue is easy to reproduce with test giving negative data set. 0 should not pass as supported attribute for any set of attributes but it does as in_array in the method does not use flag 'strict' set to true. As this is abstract voter and is used by users with their code flag 'strict' should be set to true. Since is there in 2.7 and 2.8 (LTS) IMHO it should be fixed. Commits ------- 8306530 [Security] AbstractVoter method supportsAttribute gives false positive if attribute is zero (0)
| | * [Security] AbstractVoter method supportsAttribute gives false positive if ↵Martynas Narbutas2016-12-032-1/+72
| | | | | | | | | | | | attribute is zero (0)
* | | Merge branch '2.7' into 2.8v2.8.15Nicolas Grekas2016-12-083-3/+3
|\ \ \ | |/ / | | | | | | | | | | | | | | | * 2.7: [Twig] Fix deprecations with Twig 1.29 fix the docblock in regard to the role argument Cast result to int before adding to it
| * | fix the docblock in regard to the role argumentv2.7.22Christian Flothmann2016-12-063-3/+3
| |/
* | Fix email addressVictor Bocharsky2016-12-071-1/+1
| |
* | Merge branch '2.7' into 2.8Nicolas Grekas2016-11-251-2/+2
|\ \ | |/ | | | | | | | | | | | | | | | | | | | | * 2.7: [Routing] Fail properly when a route parameter name cannot be used as a PCRE subpattern name [FrameworkBundle] Improve performance of ControllerNameParser Update documentation link to the component [HttpFoundation] Add links to RFC-7231 [DI] Initialize properties before method calls Tag missing internals [WebProfilerBundle] Dont use request attributes in RouterController Fix complete config tests
| * minor #20626 Tag missing internals (ogizanagi)Nicolas Grekas2016-11-251-2/+2
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR was merged into the 2.7 branch. Discussion ---------- Tag missing internals | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | N/A | License | MIT | Doc PR | N/A Commits ------- 97e94b4 Tag missing internals
| | * Tag missing internalsMaxime Steinhausser2016-11-251-2/+2
| | |
* | | Merge branch '2.7' into 2.8Fabien Potencier2016-11-231-1/+1
|\ \ \ | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [DI] minor FileLoaders tests update [HttpKernel] Revert BC breaking change of Request::isMethodSafe() [DOMCrawler] Bug fixed [Process] Do feat test before enabling TTY mode bumped Symfony version to 2.7.22 updated VERSION for 2.7.21 update CONTRIBUTORS for 2.7.21 updated CHANGELOG for 2.7.21 [Doctrine][Form] support large integers
| * | [HttpKernel] Revert BC breaking change of Request::isMethodSafe()Nicolas Grekas2016-11-231-1/+1
| |/
* | Merge branch '2.7' into 2.8v2.8.14Fabien Potencier2016-11-151-1/+1
|\ \ | |/ | | | | | | * 2.7: DX: replace @link with @see annotation
| * DX: replace @link with @see annotationv2.7.21Dariusz Ruminski2016-11-131-1/+1
| |
* | Merge branch '2.7' into 2.8Nicolas Grekas2016-11-031-2/+2
|\ \ | |/ | | | | | | | | | | | | | | | | * 2.7: CS fixes Remove trailing space CS: apply rules [Yaml] Clean some messages + add test case [Form] Fix UrlType transforms valid protocols [SecurityBundle] Changed encoder configuration example to bcrypt
* | Merge branch '2.7' into 2.8v2.8.13Fabien Potencier2016-10-052-1/+12
|\ \ | |/ | | | | | | | | | | | | * 2.7: [HttpFoundation] JSONP callback validation [Console] Improved the explanation of the hasOption() method add missing use statement bug #18042 [Security] $attributes can be anything, but RoleVoter assumes strings
| * bug #19725 [Security] $attributes can be anything, but RoleVoter assumes ↵v2.7.20Fabien Potencier2016-10-052-1/+12
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | strings (Jonatan Männchen) This PR was merged into the 2.7 branch. Discussion ---------- [Security] $attributes can be anything, but RoleVoter assumes strings | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes | New feature? | no | BC breaks? | yes | Deprecations? | no | Tests pass? | yes | Fixed tickets | #18042 | License | MIT | Doc PR | reference to the documentation PR, if any Commits ------- ad3ac95 bug #18042 [Security] $attributes can be anything, but RoleVoter assumes strings
| | * bug #18042 [Security] $attributes can be anything, but RoleVoter assumes stringsJonatan Männchen2016-09-212-1/+12
| | |
* | | Merge branch '2.7' into 2.8v2.8.12Fabien Potencier2016-09-242-0/+142
|\ \ \ | |/ / | | | | | | | | | | | | | | | * 2.7: Fixed expectedException annotations Security and validators messages translation to Latvian [Form] FormView->isRendered() remove dead code and simplify the flow
| * | Security and validators messages translation to Latvianv2.7.19Rikijs Murgs2016-09-222-0/+142
| |/
* | Merge branch '2.7' into 2.8Fabien Potencier2016-09-121-2/+10
|\ \ | |/ | | | | | | | | | | | | | | | | | | | | * 2.7: [travis/appveyor] Wire simple-phpunit [Console] fixed PHP7 Errors are now handled and converted to Exceptions Fix #19721 bumped Symfony version to 2.7.19 updated VERSION for 2.7.18 update CONTRIBUTORS for 2.7.18 updated CHANGELOG for 2.7.18 [Security] Optimize RoleHierarchy's buildRoleMap method
| * minor #19868 [Security] Optimize RoleHierarchy's buildRoleMap method (Enleur)v2.7.18Fabien Potencier2016-09-061-2/+10
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR was squashed before being merged into the 2.7 branch (closes #19868). Discussion ---------- [Security] Optimize RoleHierarchy's buildRoleMap method | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | no | License | MIT | Doc PR | no I have an issue with a large role hierarchy(~150 roles). Optimized it a little bit ![image](https://cloud.githubusercontent.com/assets/858989/18271257/df6c4ba0-7439-11e6-8406-e13bdcefe9ca.png) Commits ------- c3b68b0 [Security] Optimize RoleHierarchy's buildRoleMap method
| | * [Security] Optimize RoleHierarchy's buildRoleMap methodEnleur2016-09-061-2/+10
| | |
* | | minor #19773 [Security] Added note inside phpdoc. (Koc)v2.8.11Fabien Potencier2016-09-061-0/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR was squashed before being merged into the 2.8 branch (closes #19773). Discussion ---------- [Security] Added note inside phpdoc. | Q | A | ------------- | --- | Branch? | 2.8 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Commits ------- c49e462 [Security] Added note inside phpdoc.
| * | | [Security] Added note inside phpdoc.Konstantin Myakshin2016-09-061-0/+1
| | | |
* | | | Merge branch '2.7' into 2.8Nicolas Grekas2016-09-061-1/+1
|\ \ \ \ | |/ / / |/| / / | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [FrameworkBundle] Check for class existence before is_subclass_of Update GroupSequence.php Code enhancement and cleanup [DI] Add anti-regression test Revert "minor #19689 [DI] Cleanup array_key_exists (ro0NL)" [BrowserKit] Fix cookie expiration on 32 bit systems bumped Symfony version to 2.7.18 updated VERSION for 2.7.17 update CONTRIBUTORS for 2.7.17 updated CHANGELOG for 2.7.17 Update misleading comment about RFC4627
| * | Code enhancement and cleanupYonel Ceruto2016-09-061-1/+1
| | |
* | | Merge branch '2.7' into 2.8v2.8.10Nicolas Grekas2016-08-268-12/+18
|\ \ \ | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [Validator][GroupSequence] fixed GroupSequence validation ignores PropertyMetadata of parent classes [FrameworkBundle][Security] Remove useless mocks [DoctrineBridge] Enhance exception message in EntityUserProvider added friendly exception when constraint validator does not exist or it is not enabled remove duplicate instruction [FrameworkBundle] Remove TranslatorBagInterface check [FrameworkBundle] Remove duplicated code in RouterDebugCommand [Validator] fixed duplicate constraints with parent class interfaces SecurityBundle:BasicAuthenticationListener: removed a default argument on getting a header value
| * | minor #19736 [FrameworkBundle][Security] Remove useless mocks (Ener-Getick)v2.7.17Fabien Potencier2016-08-257-11/+17
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR was merged into the 2.7 branch. Discussion ---------- [FrameworkBundle][Security] Remove useless mocks | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Removes mocks causing issues in https://github.com/symfony/symfony/pull/19734. Commits ------- fcd3345 [FrameworkBundle][Security] Remove useless mocks
| | * | [FrameworkBundle][Security] Remove useless mocksEner-Getick2016-08-257-11/+17
| | | |
| * | | SecurityBundle:BasicAuthenticationListener: removed a default argument on ↵Dawid Nowak2016-08-011-1/+1
| |/ / | | | | | | | | | | | | | | | getting a header value it's unnecessary.
| * | Minor fixesv2.7.16Vyacheslav Pavlov2016-07-301-2/+0
| | |
| * | Fixed bugs in names of classes and methods.Vyacheslav Pavlov2016-07-261-1/+1
| | |
* | | Merge branch '2.7' into 2.8v2.8.9Nicolas Grekas2016-07-171-1/+7
|\ \ \ | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [VarDumper] Fix dumping jsons casted as arrays PassConfig::getMergePass is not an array Revert "bug #19114 [HttpKernel] Dont close the reponse stream in debug (nicolas-grekas)" Fix the retrieval of the last username when using forwarding [Yaml] Fix PHPDoc of the Yaml class [HttpFoundation] Add OPTIONS and TRACE to the list of safe methods Update getAbsoluteUri() for query string uris
| * | Fix the retrieval of the last username when using forwardingChristophe Coevoet2016-07-111-1/+7
| | |
* | | [Security] Fix deprecated usage of DigestAuthenticationEntryPoint::getKey() ↵Maxime STEINHAUSSER2016-07-082-1/+80
| | | | | | | | | | | | in DigestAuthenticationListener
* | | Merge branch '2.7' into 2.8Nicolas Grekas2016-07-057-15/+1
|\ \ \ | |/ / | | | | | | | | | | | | | | | | | | * 2.7: removed @since Remove and change unrelevant comments in Validator and Security components. [Validator] UuidValidator must accept a Uuid constraint. [Validator] make UuidValidator class formatting consistent.
| * | removed @sinceFabien Potencier2016-07-047-14/+0
| | |
| * | Remove and change unrelevant comments in Validator and Security components.Hugo Hamon2016-07-011-1/+1
| | |
* | | [Security][Guard] check if session exist before using itJean Pasdeloup2016-06-302-2/+222
| | |
* | | Merge branch '2.7' into 2.8v2.8.8Fabien Potencier2016-06-297-7/+7
|\ \ \ | |/ / | | | | | | | | | | | | * 2.7: removed dots at the end of @param and @return fixed typo
| * | removed dots at the end of @param and @returnv2.7.15Fabien Potencier2016-06-288-8/+8
| | |
| * | `@throws` annotations should go after `@return`v2.7.14Nicolas Grekas2016-06-065-18/+18
| | |
* | | [Security] [Guard] Improve comment with working exampleJean Pasdeloup2016-06-231-5/+9
| | |
* | | security #18736 Fixed issue with blank password with Ldap (csarrazi)v2.8.7v2.8.6Fabien Potencier2016-05-092-1/+22
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR was merged into the 2.8 branch. Discussion ---------- Fixed issue with blank password with Ldap | Q | A | ------------- | --- | Branch? | 1.8 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | n/a | License | MIT | Doc PR | n/a Commits ------- c7d9c62 Fixed issue with blank password with Ldap
| * | | Fixed issue with blank password with LdapCharles Sarrazin2016-05-092-1/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The bind operation of LDAP, as described in RFC 4513, provides a method which allows for authentication of users. For the Simple Authentication Method a user may use the anonymous authentication mechanism, the unauthenticated authentication mechanism, or the name/password authentication mechanism. The unauthenticated authentication mechanism is used when a client who desires to establish an anonymous authorization state passes a non-zero length distinguished name and a zero length password. Most LDAP servers either can be configured to allow this mechanism or allow it by default. _Web-based applications which perform the simple bind operation with the client's credentials are at risk when an anonymous authorization state is established. This can occur when the web-based application passes a distinguished name and a zero length password to the LDAP server._ Thus, misconfiguring a server with simple bind can trick Symfony into thinking the username/password tuple as valid, potentially leading to unauthorized access.
* | | | Merge branch '2.7' into 2.8Fabien Potencier2016-05-095-0/+90
|\ \ \ \ | | |/ / | |/| | | | | | | | | | * 2.7: limited the maximum length of a submitted username
| * | | Merge branch '2.3' into 2.7v2.7.13Fabien Potencier2016-05-095-0/+90
| |\ \ \ | | | | | | | | | | | | | | | | | | | | * 2.3: limited the maximum length of a submitted username
| | * | | limited the maximum length of a submitted usernamev2.3.42v2.3.41origin/2.3Fabien Potencier2016-05-093-0/+84
| | | | |
* | | | | Merge branch '2.7' into 2.8Fabien Potencier2016-05-092-2/+2
|\ \ \ \ \ | |/ / / / | | | / / | |_|/ / |/| | | | | | | | | | | | | | | | | | | * 2.7: [2.3][Component/Security] Fixed phpdoc in AnonymousToken constructor for user param prevent calling get() for service_container service call get() after the container was compiled Fixed readme of OptionsResolver [DependencyInjection] Suggest ExpressionLanguage in composer.json
| * | | Merge branch '2.3' into 2.7Fabien Potencier2016-05-092-2/+2
| |\ \ \ | | |/ / | | | | | | | | | | | | | | | | | | | | * 2.3: [2.3][Component/Security] Fixed phpdoc in AnonymousToken constructor for user param call get() after the container was compiled Fixed readme of OptionsResolver
| | * | minor #18727 [2.3][Component/Security] Fixed phpdoc in AnonymousToken ↵Fabien Potencier2016-05-092-2/+2
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | constructor for user param (eriwin) This PR was squashed before being merged into the 2.3 branch (closes #18727). Discussion ---------- [2.3][Component/Security] Fixed phpdoc in AnonymousToken constructor for user param | Q | A | ------------- | --- | Branch? | 2.3 | Bug fix? | yes, phpdoc one | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Updated phpdoc of AnonymousToken $user param from string to string|object since an object is allowed to in the parent AbstractToken: https://github.com/symfony/symfony/blob/2.3/src/Symfony/Component/Security/Core/Authentication/Token/AbstractToken.php#L91 Commits ------- b1c60b4 [2.3][Component/Security] Fixed phpdoc in AnonymousToken constructor for user param
| | | * | [2.3][Component/Security] Fixed phpdoc in AnonymousToken constructor for ↵Erik van Wingerden2016-05-092-2/+2
| | | | | | | | | | | | | | | | | | | | user param
* | | | | Merge branch '2.7' into 2.8Fabien Potencier2016-05-031-0/+4
|\ \ \ \ \ | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: add @Event annotation for AuthenticationEvents add @Event annotation for KernelEvents bumped Symfony version to 2.7.13 updated VERSION for 2.7.12 update CONTRIBUTORS for 2.7.12 updated CHANGELOG for 2.7.12 bumped Symfony version to 2.3.41 updated VERSION for 2.3.40 update CONTRIBUTORS for 2.3.40 updated CHANGELOG for 2.3.40 Revert "minor #18257 [Routing] Don't needlessly execute strtr's as they are fairly expensive (arjenm)" Revert "fixed CS" [FrameworkBundle] Remove misleading comment bug #17460 [DI] fix ambiguous services schema
| * | | | Merge branch '2.3' into 2.7Fabien Potencier2016-05-031-0/+4
| |\ \ \ \ | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.3: add @Event annotation for AuthenticationEvents bumped Symfony version to 2.3.41 updated VERSION for 2.3.40 update CONTRIBUTORS for 2.3.40 updated CHANGELOG for 2.3.40 bug #17460 [DI] fix ambiguous services schema
| | * | | add @Event annotation for AuthenticationEventsDaniel Espendiller2016-05-031-0/+4
| | | | |
* | | | | [Security] Normalize "symfony/security-acl" dependency versions across all ↵v2.8.5Shawn Iwinski2016-04-191-1/+1
| | | | | | | | | | | | | | | | | | | | composer.json files
* | | | | Merge branch '2.7' into 2.8Christian Flothmann2016-04-121-2/+7
|\ \ \ \ \ | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [Filesystem] Better error handling in remove() [DependencyInjection] Add coverage for invalid Expression in exportParameters [DependencyInjection] Add coverage for all invalid arguments in exportParameters anonymous services are always private [Console] Correct time formatting. [DependencyInjection] Resolve aliases before removing abstract services + add tests Fix Dom Crawler select option with empty value Remove unnecessary option assignment remove unused variable mock the proper method [PropertyAccess] Fix regression
| * | | | Merge branch '2.3' into 2.7v2.7.12Christian Flothmann2016-04-091-1/+0
| |\ \ \ \ | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.3: [DependencyInjection] Resolve aliases before removing abstract services + add tests Fix Dom Crawler select option with empty value Remove unnecessary option assignment remove unused variable [PropertyAccess] Fix regression
| | * | | remove unused variablev2.3.40Christian Flothmann2016-04-051-1/+0
| | | | |
| * | | | mock the proper methodChristian Flothmann2016-04-051-1/+7
| | | | |
* | | | | Merge branch '2.7' into 2.8Fabien Potencier2016-04-052-1/+50
|\ \ \ \ \ | |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [HttpFoundation] Improve phpdoc [Logging] Add support for firefox in ChromePhpHandler Windows 10 version check in just one line Detect CLI color support for Windows 10 build 10586 [Security] Fixed SwitchUserListener when exiting an impersonication with AnonymousToken [EventDispatcher] Try first if the event is Stopped [FrameworkBundle] fixes grammar in container:debug command manual. [Form] fix "prototype" not required when parent form is not required
| * | | | Merge branch '2.3' into 2.7Fabien Potencier2016-04-052-1/+50
| |\ \ \ \ | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.3: [HttpFoundation] Improve phpdoc [Logging] Add support for firefox in ChromePhpHandler [Security] Fixed SwitchUserListener when exiting an impersonication with AnonymousToken [Form] fix "prototype" not required when parent form is not required
| | * | | [Security] Fixed SwitchUserListener when exiting an impersonication with ↵Grégoire Pineau2016-04-042-1/+50
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AnonymousToken If you configure a firewall with switch user with `role: IS_AUTHENTICATED_ANONYMOUSLY` it's impossible to exit the impersonation because the next line `$this->provider->refreshUser($original->getUser())` will fail. It fails because `RefreshUser` expects an instance of `UserInterface` and here it's a string. Therefore, it does not make sense to refresh an Anonymous Token, right ?
| * | | fix mocksv2.7.11Christian Flothmann2016-03-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * check for existance of `setMetadataFactory()` method (this is needed for tests run with deps=high as the method was removed in Symfony 3.0) * fix mock testing the `EngineInterface` as the `stream()` method cannot be mocked when it is does not exist in the mocked interface
| * | | Merge branch '2.3' into 2.7Nicolas Grekas2016-03-252-12/+16
| |\ \ \ | | |/ / | | | / | | |/ | |/| | | | | | | | | | * 2.3: fix mocks Conflicts: src/Symfony/Component/Security/Http/Tests/Firewall/SwitchUserListenerTest.php
| | * fix mocksChristian Flothmann2016-03-252-12/+16
| | | | | | | | | | | | | | | * fix a typo in a method name (`setResponse` instead of `seetResopnse`) * fix mocking a method that is not part of the `DomainObjectInterface`
| | * minor #18119 [Hack Day] Issue for newcomers (2). Replace perfom by perform ↵v2.3.39Fabien Potencier2016-03-121-1/+1
| | |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (JuanLuisGarciaBorrego) This PR was merged into the 2.3 branch. Discussion ---------- [Hack Day] Issue for newcomers (2). Replace perfom by perform | Q | A | ------------- | --- | Branch | 2.3 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | no | Fixed tickets | #18089 | License | MIT | Doc PR | reference to the documentation PR, if any Commits ------- f33f192 replace perfom by perform
| | | * replace perfom by performjuanluisgarciaborrego2016-03-121-1/+1
| | | |
* | | | Updated some missing READMEsv2.8.4Javier Eguiluz2016-03-101-12/+5
| | | |
* | | | Merge branch '2.7' into 2.8Nicolas Grekas2016-03-073-36/+15
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [HttpFoundation] Fix transient test [HttpFoundation] Add a dependency on the mbstring polyfill [2.7] update readme files for new components add readme files where missing fix lowest TwigBridge deps versions [EventDispatcher] fix syntax error Don't use reflections when possible Don't use reflections when possible [Form] Update form tests after the ICU data update [Intl] Update tests and the number formatter to match behaviour of the intl extension [Intl] Update the ICU data to version 55 [Intl] Fix the update-data.php script in preparation for ICU 5.5 [Process] Fix memory issue when using large input streams Use constant instead of function call. fixed test name automatically generate safe fallback filename [Console] default to stderr in the console helpers Conflicts: composer.json src/Symfony/Bridge/PhpUnit/README.md src/Symfony/Bridge/Twig/composer.json src/Symfony/Component/Console/Helper/DialogHelper.php src/Symfony/Component/Debug/DebugClassLoader.php src/Symfony/Component/HttpFoundation/composer.json src/Symfony/Component/Security/Acl/README.md src/Symfony/Component/Security/Core/README.md src/Symfony/Component/Security/Csrf/README.md src/Symfony/Component/Security/Http/README.md
| * | | [2.7] update readme files for new componentsChristian Flothmann2016-03-074-48/+20
| | | |
* | | | Merge branch '2.7' into 2.8Fabien Potencier2016-03-041-16/+11
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: Updated all the README files [TwigBundle] Fix failing test on appveyor Improved the error message when using "@" in a decorated service Improve error reporting in router panel of web profiler [DoctrineBridge][Form] Fix performance regression in EntityType [FrameworkBundle] Fix a regression in handling absolute and namespaced template paths Allow to normalize \Traversable minor [Form] fix tests added by #16886 Remove _path from query parameters when fragment is a subrequest and request attributes are already set Added tests for _path removal in FragmentListener Simplified everything Added a test Fixed the problem in an easier way Fixed a syntax issue Improved the error message when a template is not found [CodingStandards] Conformed to coding standards [TwigBundle] fixed Include file locations in "Template could not be found" exception
| * | | Merge branch '2.3' into 2.7Fabien Potencier2016-03-041-16/+11
| |\ \ \ | | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.3: Updated all the README files [TwigBundle] Fix failing test on appveyor [FrameworkBundle] Fix a regression in handling absolute and namespaced template paths Allow to normalize \Traversable Remove _path from query parameters when fragment is a subrequest and request attributes are already set Added tests for _path removal in FragmentListener Simplified everything Added a test Fixed the problem in an easier way Fixed a syntax issue Improved the error message when a template is not found [CodingStandards] Conformed to coding standards [TwigBundle] fixed Include file locations in "Template could not be found" exception
| | * | Updated all the README filesJavier Eguiluz2016-03-041-16/+11
| | | |
* | | | Merge branch '2.7' into 2.8v2.8.3Fabien Potencier2016-02-261-0/+31
|\ \ \ \ | |/ / / | | | | | | | | | | | | * 2.7: [Security] added some tests
| * | | [Security] added some testsv2.7.10Fabien Potencier2016-02-261-0/+31
| | | |
* | | | Merge branch '2.7' into 2.8Fabien Potencier2016-02-231-0/+31
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | * 2.7: Validate XLIFF translation files [DependencyInjection] replace alias in factories replace alias in factory services
| * | | Merge branch '2.3' into 2.7Fabien Potencier2016-02-231-0/+31
| |\ \ \ | | |/ / | | | | | | | | | | | | | | | | * 2.3: Validate XLIFF translation files replace alias in factory services
| | * | Validate XLIFF translation filesv2.3.38Javier Eguiluz2016-02-231-0/+31
| | | |
* | | | Merge branch '2.7' into 2.8Nicolas Grekas2016-02-221-2/+2
|\ \ \ \ | |/ / / | | | | | | | | | | | | * 2.7: Fix merge
| * | | Fix mergeNicolas Grekas2016-02-221-2/+2
| | | |
* | | | Merge branch '2.7' into 2.8Nicolas Grekas2016-02-221-2/+2
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | * 2.7: PhpUnitNoDedicateAssertFixer results Improve Norwegian translations Simplify markdown for PR template
| * | | Merge branch '2.3' into 2.7Nicolas Grekas2016-02-221-2/+2
| |\ \ \ | | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.3: PhpUnitNoDedicateAssertFixer results Improve Norwegian translations Simplify markdown for PR template Conflicts: src/Symfony/Bridge/Propel1/Tests/Form/DataTransformer/CollectionToArrayTransformerTest.php src/Symfony/Component/Security/Tests/Core/SecurityContextTest.php
| | * | PhpUnitNoDedicateAssertFixer resultsSpacePossum2016-02-211-1/+1
| | | |
| | * | Improve Norwegian translationsValentin Jonovs2016-02-201-2/+2
| | | |
* | | | Merge branch '2.7' into 2.8Nicolas Grekas2016-02-021-0/+9
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: [Console] Add missing `@require` annotation in test Fix merge [appveyor] Fix failure reporting [#17634] move DebugBundle license file backport GlobTest from 2.7 branch Move licenses according to new best practices [FrameworkBundle] Remove unused code in test [2.3] Fixed an undefined variable in Glob::toRegex simplified a test fix container cache key generation [Form] fix option name in changelog [Translation] Add resources from fallback locale [DependencyInjection] enforce tags to have a name [YAML] Refine the return value of Yaml::parse() Conflicts: src/Symfony/Component/DependencyInjection/Tests/Loader/YamlFileLoaderTest.php
| * | | Merge branch '2.3' into 2.7Nicolas Grekas2016-02-011-0/+9
| |\ \ \ | | |/ / | | | | | | | | | | | | * 2.3: [appveyor] Fix failure reporting
| | * | [appveyor] Fix failure reportingNicolas Grekas2016-02-011-0/+9
| | | |
* | | | Merge branch '2.7' into 2.8Fabien Potencier2016-01-271-2/+1
|\ \ \ \ | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 2.7: Increase the inlining YAML level for config:debug [Serializer] Minor: fix CS and PHPDoc [Form] fix tests [Serializer] Ensure that groups are strings [Debug] Tell that the extension is for PHP 5 only Static code analysis Update AnnotationDirectoryLoader.php added a test Escape the delimiter in Glob::toRegex [FrameworkBundle] Fix template location for PHP templates [FrameworkBundle] Add path verification to the template parsing test cases
| * | | Merge branch '2.3' into 2.7Fabien Potencier2016-01-272-3/+2
| |\ \ \ | | |/ / | | | | | | | | | | | | | | | | | | | | | | | | * 2.3: Static code analysis Update AnnotationDirectoryLoader.php [FrameworkBundle] Fix template location for PHP templates [FrameworkBundle] Add path verification to the template parsing test cases
| | * | minor #17480 Static code analysis (Koc)Fabien Potencier2016-01-273-4/+3
| | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This PR was merged into the 2.3 branch. Discussion ---------- Static code analysis | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - Things that done: * fix case in method calls * removed unused imports * use shorter concat where it possible * optimize some css * removed duplicated array keys * removed redurant return statements * removed one-time variables * do not pass arguments that not used in functions Commits ------- 8db691a Static code analysis
| | | * | Static code analysisKonstantin.Myakshin2016-01-253-4/+3
| | | | |
generated by cgit v1.1 at 2025-05-10 22:23:44 +0000