-
Notifications
You must be signed in to change notification settings - Fork 95
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
Added Level of Assurance (LoA) section; fixes #151; fixes #391 #568
Conversation
This also addresses #391 |
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.
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
@peacekeeper I will add FIDO, FIDO 2 / WebAuthn etc. as examples for SCA. I could also add examples for NIST, ISO, eIDAS LoA frameworks if useful. |
(probably worth also adjusting title of this PR, espexcially typo |
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.
Minor nits, otherwise LGTM, thanks @awoie! :)
Co-authored-by: Manu Sporny <[email protected]>
Co-authored-by: Manu Sporny <[email protected]>
I added some references to NIST, ISO, FIDO/Webauthn. |
Maybe @Oskar-van-Deventer also wants to review this, since his issue #151 was the original inspiration for addressing this topic in the spec. |
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
Co-authored-by: Ted Thibodeau Jr <[email protected]>
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.
This probably should also say that Verifiable Credentials are an appropriate way to convey this information as well. We can do this editorially or in a future PR.
Editorial, multiple reviews, changes requested and made, no objections, merging. |
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.
+1 to merge
As per @peacekeeper request, I tried to come up with a PR that covers LoAs; fixes #151
Preview | Diff