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
Detect expired entries on read #41
Conversation
When reading sessions that should be expired expire them Also fixes memory Init() to make the Gitea copy of the memory provider unnecessary. Signed-off-by: Andrew Thornton <art27@cantab.net> Reviewed-on: https://gitea.com/macaron/session/pulls/11 Reviewed-by: 6543 <6543@noreply.gitea.io> Reviewed-by: techknowlogick <techknowlogick@gitea.io>
Update 'postgres/postgres.go' Reviewed-on: https://gitea.com/macaron/session/pulls/12
@unknwon the expire enhancements updated :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR!
Can you add comments on every new addition about rationale? It is not clear to me what has been improved or fixed.
Co-authored-by: zeripath <art27@cantab.net>
@unknwon does #41 (comment) explain it :) ? |
Thanks for the follow up! One last minor issue. |
@unknwon you are right, changed :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
https://github.com/go-macaron/session/releases/tag/v1.0.1 has been published for this merge. |
When reading sessions that should be expired expire them
Also fixes memory Init()