-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
session id validation when using php7.1 #371
Comments
Adapter, redis etc. are have bugs. |
Can you guys provide specifics about what you're seeing happening, what your setup is, etc. I need more details than, "It's not working" :) |
Hi :) reference ; |
@byazrail That's great. Thanks. Could you submit that as a pull request? I did see something about those issues coming to light, but didn't follow it close enough, obviously. :) |
@lonnieezell welcome, submitted as pull request. |
Regex: /^[0-9a-f]{40}$/ is not working correctly when using php 7.1. According to php manual (http://php.net/manual/en/session.configuration.php#ini.session.sid-length), these values should be set as follow
session.sid_length = 40
session.sid_bits_per_character = 4
In my case, session file regenerated for every request, when FileHandler was used as session driver, and
session.sid_length = 32
session.sid_bits_per_character = 5
The text was updated successfully, but these errors were encountered: