-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[HttpFoundation] Redis session handler #14539
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
Conversation
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Tests pass? | yes |
Fixed tickets | - |
License | MIT |
Doc PR | symfony/symfony-docs#5230 |
I hope this gets in on 2.7 though it has been feature freezed 😟 |
I'm pretty sure there already exists a redis session handler, even as PHP C extension. |
@Tobion please have a closer look. This is just a wrapper which internally uses the C extension. You can set the session handler on a PHP configuration level but often it is more convenient to place it right into your app. That's why all other handlers were created in the first place. |
@sgrodzicki I had a close lock and I don't see the point of it.
So what's the advantage? |
When do you think it's more convenient to use a custom handler, instead of the native one? |
@jakzal when you don't have access to the PHP configuration file and/or you're hosting multiple Symfony projects on the same server/workers. |
@sgrodzicki you can set the native handler in code with ini_set. You don't need access to the config file. |
@Tobion how is that different from Memcached/MongoDB? |
Closing for the reasons given by @Tobion |
This PR was merged into the 4.1-dev branch. Discussion ---------- [HttpFoundation] RedisSessionHandler | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #24433, #18233, #14539, #4538, #3498 | License | MIT | Doc PR | symfony/symfony-docs#8572 Ability to use Redis as a session storage backend. Discussed in detail in linked issues / PRs. Commits ------- 8776cce [HttpFoundation] Add RedisSessionHandler