Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Allow the 'gc_probability' option to be set to zero. #4608

Closed
wants to merge 1 commit into from
Closed

Allow the 'gc_probability' option to be set to zero. #4608

wants to merge 1 commit into from

Conversation

gws
Copy link
Contributor

@gws gws commented Jun 8, 2013

This is valid because PHP divides gc_divisor into gc_probability - it would only
be invalid to set gc_divisor to 0 (divide-by-zero), so this commit leaves the
same test for that option alone.

This is valid because PHP divides gc_divisor into gc_probability - it would only
be invalid to set gc_divisor to 0 (divide-by-zero), so this commit leaves the
same test for that option alone.
@gws
Copy link
Contributor Author

gws commented Jun 8, 2013

It's my understanding that this is the "PHP way" of disabling session garbage collection which is really what I'm trying to do - so if there's a "ZF2 way" I missed I'll happily close this.

@ghost ghost assigned weierophinney Jun 11, 2013
@weierophinney
Copy link
Member

@Gwis nope, this is perfect -- the session configuration is meant to proxy to PHP's session INI settings! :)

weierophinney added a commit that referenced this pull request Jun 11, 2013
Allow the 'gc_probability' option to be set to zero.
weierophinney added a commit that referenced this pull request Jun 11, 2013
weierophinney added a commit to zendframework/zend-session that referenced this pull request May 15, 2015
…ero-gc-probability

Allow the 'gc_probability' option to be set to zero.
weierophinney added a commit to zendframework/zend-session that referenced this pull request May 15, 2015
weierophinney added a commit to zendframework/zend-session that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants