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

Skip to content

Fix use_cookies framework session configuration #45964

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
Apr 8, 2022

Conversation

alexander-schranz
Copy link
Contributor

@alexander-schranz alexander-schranz commented Apr 7, 2022

Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #45105
License MIT
Doc PR symfony/symfony-docs#...

Keep use_cookies false flag in mind.

Todo:

  • test case

@carsonbot
Copy link

Hey!

To help keep things organized, we don't allow "Draft" pull requests. Could you please click the "ready for review" button or close this PR and open a new one when you are done?

Note that a pull request does not have to be "perfect" or "ready for merge" when you first open it. We just want it to be ready for a first review.

Cheers!

Carsonbot

@alexander-schranz alexander-schranz marked this pull request as ready for review April 7, 2022 01:03
@carsonbot carsonbot added this to the 5.4 milestone Apr 7, 2022
@alexander-schranz alexander-schranz changed the title Fix use_cookies framework session configuration [HttpKernel] Fix use_cookies framework session configuration Apr 7, 2022
@carsonbot carsonbot changed the title [HttpKernel] Fix use_cookies framework session configuration Fix use_cookies framework session configuration Apr 7, 2022
@fabpot
Copy link
Member

fabpot commented Apr 8, 2022

Thank you @alexander-schranz.

@fabpot fabpot merged commit dcd1c32 into symfony:5.4 Apr 8, 2022
@alexander-schranz alexander-schranz deleted the patch-10 branch April 8, 2022 08:16
This was referenced Apr 27, 2022
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.

4 participants