Skip to content

[Session] Add docs to define redis session handler via php.ini directly #17399

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Oct 26, 2022
Merged

[Session] Add docs to define redis session handler via php.ini directly #17399

merged 1 commit into from
Oct 26, 2022

Conversation

alexander-schranz
Copy link
Contributor

@alexander-schranz alexander-schranz commented Oct 25, 2022 •
edited
Loading

I think the most easiest way to configure session handling via redis is still configure it via php.ini and not via the Symfony Application.

@carsonbot carsonbot added this to the 4.4 milestone Oct 25, 2022
@alexander-schranz alexander-schranz changed the title Add possibility to define redis session handler via php.ini directly Add docs to define redis session handler via php.ini directly Oct 25, 2022
@carsonbot carsonbot changed the title Add docs to define redis session handler via php.ini directly [Session] Add docs to define redis session handler via php.ini directly Oct 26, 2022
@javiereguiluz javiereguiluz merged commit a3ece80 into symfony:4.4 Oct 26, 2022
@javiereguiluz
Copy link
Member

Alexander, thanks for this contribution.

While merging I did some minor tweaks (2c59168) to ensure that it's perfectly clear that there are two possible options; both are different and independent, so readers must choose and follow only one of them.

@alexander-schranz alexander-schranz deleted the patch-1 branch October 26, 2022 13:44
@alexander-schranz
Copy link
Contributor Author

Thx sounds great and make it more clear 👍

@alexander-schranz
Copy link
Contributor Author

@javiereguiluz looks like the numbering did fail not sure how to fix that: https://symfony.com/doc/current/session/database.html#store-sessions-in-a-key-value-database-redis

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Session Status: Reviewed
Projects
None yet
Milestone
4.4
Development

Successfully merging this pull request may close these issues.
3 participants
@alexander-schranz @javiereguiluz @carsonbot