Commit ec1e4a2
committed
bug symfony#60379 [Security] Avoid failing when PersistentRememberMeHandler handles a malformed cookie (Seldaek)
This PR was squashed before being merged into the 6.4 branch.
Discussion
----------
[Security] Avoid failing when PersistentRememberMeHandler handles a malformed cookie
| Q | A
| ------------- | ---
| Branch? | 6.4
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Issues | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->
| License | MIT
If the remember me cookie is malformed like `"foo"` then the page crashes due to https://github.com/symfony/symfony/blob/7.3/src/Symfony/Component/Security/Http/RememberMe/RememberMeDetails.php#L39
Not a huge deal but not very elegant
Commits
-------
2eaa7ee [Security] Avoid failing when PersistentRememberMeHandler handles a malformed cookieFile tree
2 files changed
+22
-1
lines changed- src/Symfony/Component/Security/Http
- RememberMe
- Tests/RememberMe
2 files changed
+22
-1
lines changedLines changed: 6 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
160 | 160 | | |
161 | 161 | | |
162 | 162 | | |
163 | | - | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
164 | 169 | | |
165 | 170 | | |
166 | 171 | | |
| |||
Lines changed: 16 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
74 | 74 | | |
75 | 75 | | |
76 | 76 | | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
77 | 93 | | |
78 | 94 | | |
79 | 95 | | |
| |||
0 commit comments