-
-
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 latest and session related consents #3043
base: master
Are you sure you want to change the base?
Conversation
1e65401
to
d703604
Compare
Codecov Report
@@ Coverage Diff @@
## master #3043 +/- ##
==========================================
+ Coverage 76.85% 77.16% +0.31%
==========================================
Files 124 124
Lines 9162 9348 +186
==========================================
+ Hits 7041 7213 +172
- Misses 1672 1680 +8
- Partials 449 455 +6
... and 1 file 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. |
41326c0
to
72b8cba
Compare
72b8cba
to
37adf09
Compare
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.
Can you please make this PR against the #2796 branch as we have done quite a lot of work on the DB layer and don't want to backport new changes to it as it has grown in scope significantly already. Thank you :)
37adf09
to
3c07b1c
Compare
0f125cc
to
1933a0c
Compare
d2897e2
to
c1e84a8
Compare
42d2ae7
to
70e1e72
Compare
70e1e72
to
8386cdc
Compare
47cbb3c
to
23621f4
Compare
23621f4
to
fe7957a
Compare
fe7957a
to
1f26dc6
Compare
1f26dc6
to
e6963be
Compare
This pull request introduces feature to extend consent validity time for both session related consent and subject related consent. Currently consent validity cannot be updated (when skip=true) or to be more precise - consent updates only if new device/browser accepts consent.
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security. vulnerability, I
confirm that I got green light (please contact
security@ory.sh) from the maintainers to push
the changes.
works.