diff options
author | Fabien Potencier <fabien.potencier@gmail.com> | 2015-09-27 11:53:56 +0200 |
---|---|---|
committer | Fabien Potencier <fabien.potencier@gmail.com> | 2015-09-27 11:53:56 +0200 |
commit | 6245c6234249410e049cbf6eb925941af0a704c8 (patch) | |
tree | ddde0ff4a91838225458400e7fae89acd9c79b40 /Core/Tests/Exception/CustomUserMessageAuthenticationExceptionTest.php | |
parent | 373843320ff0590eefdbff5653d758e04dcd4ddd (diff) | |
parent | 6d374205adaae6fe3886fcf888b58f06543bb7ee (diff) | |
download | symfony-security-6245c6234249410e049cbf6eb925941af0a704c8.zip symfony-security-6245c6234249410e049cbf6eb925941af0a704c8.tar.gz symfony-security-6245c6234249410e049cbf6eb925941af0a704c8.tar.bz2 |
bug #15925 Updating behavior to not continue after an authenticator has set the response (weaverryan)
This PR was merged into the 2.8 branch.
Discussion
----------
Updating behavior to not continue after an authenticator has set the response
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | https://github.com/symfony/symfony/pull/14673/files#r40492765
| License | MIT
| Doc PR | n/a
This mirrors the behavior in core: *if* a listener sets a response (on success or failure),
then the other listeners are not called. But if a response is *not* set
(which is sometimes the case for success, like in BasicAuthenticationListener),
then the other listeners are called, and can even fail.
It's all a bit of an edge-case, as only one authenticator (like authentication listener) would normally be doing any work on a request, but I think matching the other listeners (since I'm not aware of anyone having issues with its behavior) is best.
Commits
-------
5fa2684 Making all "debug" messages use the debug router
f403444 Updating behavior to not continue after an authenticator has set the response
Diffstat (limited to 'Core/Tests/Exception/CustomUserMessageAuthenticationExceptionTest.php')
0 files changed, 0 insertions, 0 deletions