Thanks for the PR. This looks like a new feature to me. As such, that's for 5.4.
Looking at other handlers, they would too benefit from options.
Also,MemcachedSessionHandler usesexpiretime instead ofttl and defaults to 86400 instead ofini_get('session.gc_maxlifetime') 🤦
Can you please have a look and try to add some consistency there?
Don't forget to update the CHANGELOG of the component while doing so.
Thanks!

Thanks for the PR. This looks like a new feature to me. As such, that's for 5.4.
Looking at other handlers, they would too benefit from options.
Also,MemcachedSessionHandler usesexpiretime instead ofttl and defaults to 86400 instead ofini_get('session.gc_maxlifetime') 🤦
Can you please have a look and try to add some consistency there?
Don't forget to update the CHANGELOG of the component while doing so.
Thanks!

@nicolas-grekas thank you for the feedback. I updated the PR to let other handlers benefit from options.

@makraz I force-pushed some changes on your fork. Could you please fetch it and add some test cases?

@makraz, I force-pushed some changes on your fork. Could you please fetch it and add some test cases?

Thank you,@nicolas-grekas, for the update. I will work on the test cases

Friendly ping@makraz (rebase needed also)

Thank you@makraz.

Btw, PR adding tests + another adding doc would be really welcomed.

Btw, PR adding tests + another adding doc would be really welcomed.

Thanks@nicolas-grekas for merging my PR, and sure I will handle it 🙌