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.

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
Oct 26, 2022

Conversation

alexander-schranz
Copy link
Contributor

@alexander-schranz alexander-schranz commented Oct 25, 2022

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
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants