-
Notifications
You must be signed in to change notification settings - Fork 268
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
ffi: add high level method for resetting the user's identity and deleting all associated secrets #3759
Conversation
f34d738
to
9b4bf7d
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3759 +/- ##
==========================================
+ Coverage 84.07% 84.09% +0.01%
==========================================
Files 259 259
Lines 27080 27098 +18
==========================================
+ Hits 22767 22787 +20
+ Misses 4313 4311 -2 ☔ View full report in Codecov by Sentry. |
35c86ad
to
c47ac22
Compare
…ting all associated secrets
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.
Alright, this looks good. Feel free to get rid of the fixup commits and merge.
68ea42b
to
ef2140d
Compare
…ication - disable backups and recovery before requesting the reset handle - attempt device key upload - re-enable backups both when UIAA is required and when not
ef2140d
to
8895e53
Compare
This PR adds support for completely resetting the user's identity and all their secrets. It will go through the following steps:
See if creating a new secrets store is necessaryNot neededCheck coverage to see if we need an OIDC specific test as well or whether the existing cross-singing tests sufficeLooks good