-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat: extend session lifespan on session refresh (skip=true) #3464
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3464 +/- ##
==========================================
+ Coverage 73.91% 76.83% +2.91%
==========================================
Files 123 123
Lines 9553 9125 -428
==========================================
- Hits 7061 7011 -50
+ Misses 2040 1666 -374
+ Partials 452 448 -4
... and 11 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
Please add a migration test file for the new migration :)
persistence/sql/migrations/20220914000001000000_support_refresh_remember_for.sqlite.up.sql
Outdated
Show resolved
Hide resolved
The access token type (`jwt` or `opaque`) can now be set in the client configuration. The value set here will overwrite the global value for all flows concerning that client.
@aeneasr Added migration test and fixed comments, please have a look! |
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.
Great job, thank you very much!
It is now possible to extend session lifespans when accepting login challenges. Closes ory#1690 Closes ory#1557 Closes ory#2246 Closes ory#2848 Co-authored-by: Mart Aarma <mart.aarma@nortal.com> Co-authored-by: Henning Perl <henning.perl@gmail.com> Co-authored-by: ory-bot <60093411+ory-bot@users.noreply.github.com>
Related issue(s)
Picking up work done in #2848 to speed up the delivery, since the author is on vacation.
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security vulnerability, I
confirm that I got the approval (please contact
security@ory.sh) from the maintainers to push
the changes.
works.
Further Comments