Skip to content

[HttpFoundation][Lock] Connection parameter corrected #34317

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

Conversation

wimg
Copy link
Contributor

@wimg wimg commented Nov 11, 2019

Q A
Branch? master
Bug fix? yes
New feature? no
Deprecations? no
Tickets
License MIT
Doc PR

Connection parameter incorrectly specified. This variable doesn't exist, the data is in $connection instead.

@nicolas-grekas nicolas-grekas changed the title Connection parameter corrected [HttpFoundation][Lock] Connection parameter corrected Nov 11, 2019
@nicolas-grekas nicolas-grekas added this to the 4.4 milestone Nov 11, 2019
@nicolas-grekas nicolas-grekas changed the base branch from master to 4.4 November 11, 2019 13:09
@nicolas-grekas
Copy link
Member

Thank you @wimg.

@nicolas-grekas nicolas-grekas force-pushed the Bugfix-SessnionHandlerFactory-this-storage branch from 811874d to 27207c3 Compare November 11, 2019 13:09
nicolas-grekas added a commit that referenced this pull request Nov 11, 2019
This PR was submitted for the master branch but it was merged into the 4.4 branch instead.

Discussion
----------

[HttpFoundation][Lock] Connection parameter corrected

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

Connection parameter incorrectly specified. This variable doesn't exist, the data is in $connection instead.

Commits
-------

27207c3 [HttpFoundation][Lock] Connection parameter corrected
@nicolas-grekas nicolas-grekas merged commit 27207c3 into symfony:4.4 Nov 11, 2019
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