-
-
Notifications
You must be signed in to change notification settings - Fork 85
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
ini_set('session.cookie_lifetime', 0); #5
Comments
"To clear any confusions that may arise: in reality, session.gc_maxlifetime |
I see that, but if this class is supposed to serve as a drop-in replacement for PHP's built-in session handling shouldn't that line not be there? What is the purpose of |
Agree with @mfurlend. To use this (wonderful) library we need to fork and change that line of code. Even if we have I don't think it's a valid assumption that site owners typically want sessions to end when a browser is closed. |
This is now fixed by a01bdee |
On lines 235-237 you have this code:
That statement does not make "session cookies never expire." It makes the cookies expire when the browser is closed. In my case this caused authenticated users to be logged off upon termination of the session/browser.
Is this a bug or the intended behavior?
The text was updated successfully, but these errors were encountered: