Skip to content
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

Php-redis driver setTimeout has deprecated #5819

Closed
al-ramadhan opened this issue Aug 19, 2019 · 6 comments
Closed

Php-redis driver setTimeout has deprecated #5819

al-ramadhan opened this issue Aug 19, 2019 · 6 comments
Labels
Milestone

Comments

@al-ramadhan
Copy link

al-ramadhan commented Aug 19, 2019

Message: Function Redis::setTimeout() is deprecated

Filename: drivers/Session_redis_driver.php

After read their official repository, turns out method "setTimeout" has been replaced with "expire" on this PR. Just tried to change drivers code and it works

@dhtmdgkr123
Copy link

Message: Function Redis::setTimeout() is deprecated
Filename: drivers/Session_redis_driver.php

After read their official repository, turns out method "setTimeout" has been replaced with "expire" on this PR. Just tried to change drivers code and it works

I have same issue so i tried to replace "setTimeout" to "expire" but i almost have

ini_set(): A session is active. You cannot change the session module's ini settings at this time
session_start(): Failed to read session data: user (path: /var/lib/php/sessions)
this error message.

my redis server is alive but can not load redis..
how can fix this issue...?

Spec : Ubuntu16.04, NGINX(1.10.3), REDIS(3.0.6), php 7.3.8, php-redis

@al-ramadhan
Copy link
Author

Maybe that's another issue even it is likely from the same redis driver. I don't know if it's related to new version of phpdriver, but looks like redis_driver has broken.

@al-ramadhan
Copy link
Author

Should be solved by this PR 5816

@dhtmdgkr123
Copy link

Should be solved by this PR 5816

Thank you~

@mchobbylong
Copy link
Contributor

Hi there! Just for reminder, the PR was merged. You could check that out 😄

@al-ramadhan
Copy link
Author

Thank you for your contribution @mchobbylong

@narfbg narfbg added the Bug label Sep 18, 2019
@narfbg narfbg added this to the 3.1.11 milestone Sep 18, 2019
@narfbg narfbg closed this as completed Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants