Skip to content

Conversation

@dkarlovi
Copy link
Contributor

@dkarlovi dkarlovi commented Nov 1, 2017

Note about RedisSessionHandler.

@dkarlovi dkarlovi force-pushed the http-foundation-redis-session-handler branch from 608aa16 to 643b9ad Compare November 1, 2017 11:35
@xabbuh xabbuh added the On hold label Nov 7, 2017
@javiereguiluz javiereguiluz added Waiting Code Merge Docs for features pending to be merged and removed On hold labels Jan 3, 2018
@nicolas-grekas
Copy link
Member

PR merged on symfony/symfony's side

@javiereguiluz javiereguiluz added Status: Reviewed and removed Waiting Code Merge Docs for features pending to be merged Status: Needs Review labels Feb 4, 2018
@javiereguiluz javiereguiluz added this to the 4.1 milestone Feb 4, 2018
nicolas-grekas added a commit to symfony/symfony that referenced this pull request Feb 4, 2018
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
@javiereguiluz
Copy link
Member

Thanks @dkarlovi.

@javiereguiluz javiereguiluz merged commit 643b9ad into symfony:master Feb 4, 2018
javiereguiluz added a commit that referenced this pull request Feb 4, 2018
… (dkarlovi)

This PR was merged into the master branch.

Discussion
----------

Introduced RedisSessionHandler as a Redis session storage

Note about RedisSessionHandler.

Commits
-------

643b9ad Introduced RedisSessionHandler as a Redis session storage
@dkarlovi dkarlovi deleted the http-foundation-redis-session-handler branch February 7, 2018 15:38
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.

5 participants