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

Fix saving sessions in PHP 5.4 with user session handlers (fix #12) #26

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
5 participants
@blino
Copy link

blino commented Mar 31, 2013

When session storage functions are set with session_set_save_handler()
(this is the "mod_user" mode), mod_data will be NULL in PHP 5.4, and
suhosin session hooks will bail out.
PHP 5.4 allows to check this with mod_user_implemented instead.

Fix saving sessions in PHP 5.4 with user session handlers (fix #12)
When session storage functions are set with session_set_save_handler()
(this is the "mod_user" mode), mod_data will be NULL in PHP 5.4, and
suhosin session hooks will bail out.
PHP 5.4 allows to check this with mod_user_implemented instead.

@blino blino referenced this pull request Mar 31, 2013

Closed

Full PHP 5.4.x compatibility #14

@NewEraCracker

This comment has been minimized.

Copy link
Contributor

NewEraCracker commented Apr 1, 2013

I can confirm your fix works in my installation. Thank you.

@ghost

This comment has been minimized.

Copy link

ghost commented Apr 9, 2013

Hi,

thank you very much. I can confirm that this has fixed the referenced issue above.

@Ressy66

This comment has been minimized.

Copy link

Ressy66 commented Jul 12, 2013

I guess its now dead anyway....
Received today, relevant bits are:

The PHP development team announces the immediate availability of PHP 5.3.27

Please Note: This will be the last regular release of the PHP 5.3
series. All users of PHP are encouraged to upgrade to PHP 5.4 or PHP
5.5.

@cryptomilk

This comment has been minimized.

Copy link

cryptomilk commented Dec 26, 2013

Could this be fixed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment