Skip to content

Commit 2ccafaf

Browse files
author
Robin Chalas
committed
minor symfony#23802 [Security] Fix security.interactive_login event const doc block (ogizanagi)
This PR was merged into the 2.7 branch. Discussion ---------- [Security] Fix security.interactive_login event const doc block | Q | A | ------------- | --- | Branch? | 2.7 <!-- see comment below --> | Bug fix? | no | New feature? | no <!-- don't forget updating src/**/CHANGELOG.md files --> | BC breaks? | no | Deprecations? | no <!-- don't forget updating UPGRADE-*.md files --> | Tests pass? | yes | Fixed tickets | N/A <!-- #-prefixed issue number(s), if any --> | License | MIT | Doc PR | N/A I'd suggest to reuse the explanation we give about this event [on the docs](http://symfony.com/doc/2.7/components/security/authentication.html#security-events) because the current one in the code is misleading: this event is not triggered for http basic/digest authentication for instance. Commits ------- f6c83cf [Security] Fix security.interactive_login event const doc block
2 parents ffa005c + f6c83cf commit 2ccafaf

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

src/Symfony/Component/Security/Http/SecurityEvents.php

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,11 @@
1414
final class SecurityEvents
1515
{
1616
/**
17-
* The INTERACTIVE_LOGIN event occurs after a user is logged in
18-
* interactively for authentication based on http, cookies or X509.
17+
* The INTERACTIVE_LOGIN event occurs after a user has actively logged
18+
* into your website. It is important to distinguish this action from
19+
* non-interactive authentication methods, such as:
20+
* - authentication based on your session.
21+
* - authentication using a HTTP basic or HTTP digest header.
1922
*
2023
* The event listener method receives a
2124
* Symfony\Component\Security\Http\Event\InteractiveLoginEvent instance.

0 commit comments

Comments
 (0)