Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Security] Revert stateless check for ContextListener #58002

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

VincentLanglet
Copy link
Contributor

@VincentLanglet VincentLanglet commented Aug 13, 2024

This reverts commit 40341a1.

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix ##57851
License MIT

Closes #57854

Cf #57854 (comment)

@carsonbot carsonbot added this to the 5.4 milestone Aug 13, 2024
@VincentLanglet VincentLanglet changed the title Revert https://github.com/symfony/symfony/pull/57372 [HttpKernel][Security] REVERT Fix accessing session for stateless request Aug 13, 2024
@VincentLanglet VincentLanglet changed the title [HttpKernel][Security] REVERT Fix accessing session for stateless request [HttpKernel][Security] Revert stateless check for ContextListener.php Aug 13, 2024
@VincentLanglet VincentLanglet changed the title [HttpKernel][Security] Revert stateless check for ContextListener.php [HttpKernel][Security] Revert stateless check for ContextListener Aug 13, 2024
@stof stof changed the title [HttpKernel][Security] Revert stateless check for ContextListener [Security] Revert stateless check for ContextListener Aug 13, 2024
@VincentLanglet VincentLanglet requested a review from stof August 14, 2024 08:56
@VincentLanglet
Copy link
Contributor Author

I cannot remove the deprecation label.

@carsonbot carsonbot changed the title [Security] Revert stateless check for ContextListener Revert stateless check for ContextListener Aug 14, 2024
@carsonbot carsonbot changed the title Revert stateless check for ContextListener [Security] Revert stateless check for ContextListener Aug 14, 2024
@fabpot
Copy link
Member

fabpot commented Aug 14, 2024

Thank you @VincentLanglet.

@fabpot fabpot merged commit 14b1b4d into symfony:5.4 Aug 14, 2024
12 checks passed
This was referenced Aug 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants