Skip to content

[HttpFoundation] Add missing session.lazy_write config option #24494

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 9, 2017

Conversation

nicolas-grekas
Copy link
Member

Q A
Branch? 2.7
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

See http://php.net/manual/en/session.configuration.php#ini.session.lazy-write

@javiereguiluz
Copy link
Member

This option was introduced in PHP 7.0 (http://php.net/manual/en/migration70.new-features.php). Is there any problem by adding this to the Symfony 2.7 branch?

@nicolas-grekas
Copy link
Member Author

The list already contains 7.1-only options, so I'd say no, there is no pb.

@fabpot
Copy link
Member

fabpot commented Oct 9, 2017

Thank you @nicolas-grekas.

@fabpot fabpot merged commit dbc9a44 into symfony:2.7 Oct 9, 2017
fabpot added a commit that referenced this pull request Oct 9, 2017
…ion (nicolas-grekas)

This PR was merged into the 2.7 branch.

Discussion
----------

[HttpFoundation] Add missing session.lazy_write config option

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

See http://php.net/manual/en/session.configuration.php#ini.session.lazy-write

Commits
-------

dbc9a44 [HttpFoundation] Add missing session.lazy_write config option
@nicolas-grekas nicolas-grekas deleted the lazy-write branch October 12, 2017 09:26
This was referenced Nov 10, 2017
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.

6 participants