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.

Sign up for GitHub

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

Jump to bottom

Remove SessionManager::rememberMe #13

Open
weierophinney opened this issue Dec 31, 2019 · 1 comment
Open

Remove SessionManager::rememberMe #13

weierophinney opened this issue Dec 31, 2019 · 1 comment
Labels
Awaiting Maintainer Response Question Further information is requested

Comments

@weierophinney
Copy link
Member

See #59 for discussion.
Also zendframework/zendframework#4775

IMO this vestige of ZF1 no longer serves any useful purpose.

Originally posted by @adamlundrigan at zendframework/zend-session#60
@weierophinney weierophinney added Awaiting Maintainer Response Question Further information is requested labels Dec 31, 2019
@weierophinney weierophinney mentioned this issue Dec 31, 2019
Open
@criterion9
Copy link

Unless I'm using the library incorrectly I don't think removing SessionManager::rememberMe is the correct action. The other discussions are around removing configuration options that probably aren't utilized during session construction. I use ::rememberMe to extend active session cookie time to limit session duration during inactive periods. If this isn't the "correct" way to handle time limited sessions with active time extensions I would love to see/hear the proper way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Awaiting Maintainer Response Question Further information is requested
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@weierophinney @criterion9