Commit 17f4fe4
committed
bug symfony#50582 [Security/Http] Fix false-string handling in
This PR was merged into the 5.4 branch.
Discussion
----------
[Security/Http] Fix false-string handling in `RememberMeAuthenticator`
| Q | A
| ------------- | ---
| Branch? | 5.4
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Tickets | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License | MIT
| Doc PR | symfony/symfony-docs#... <!-- required for new features -->
I found some errors "No remember-me cookie is found." in my logs. I didn't find another way to reproduce it other than set "false-string" to the cookie value, for example `0`.
This PR fixes this.
Commits
-------
87c2bc2 [Security] Fix false-string handling in RememberMeAuthenticatorRememberMeAuthenticator (ossinkine)File tree
2 files changed
+4
-1
lines changed- src/Symfony/Component/Security/Http
- Authenticator
- Tests/Authenticator
2 files changed
+4
-1
lines changedLines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
70 | 70 | | |
71 | 71 | | |
72 | 72 | | |
73 | | - | |
| 73 | + | |
74 | 74 | | |
75 | 75 | | |
76 | 76 | | |
| |||
Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
64 | 67 | | |
65 | 68 | | |
66 | 69 | | |
| |||
0 commit comments