Skip to content
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

Add Decoding WebAuthn blog post #153

Closed
wants to merge 10 commits into from
Closed

Add Decoding WebAuthn blog post #153

wants to merge 10 commits into from

Conversation

nielthiart
Copy link

@nielthiart nielthiart commented Dec 13, 2023

Note to editor: You'll need to run the site to see the sequence diagrams and flowchart. I've attached screenshots here for reference.

TODO:

  • Create header image
  • Fix publish date
  • Check front matter

Chart one: Components
CleanShot 2023-12-13 at 08 50 30@2x

Chart two: FIDO2 Registration
CleanShot 2023-12-13 at 08 52 18@2x

Chart three: FIDO2 Authentication
CleanShot 2023-12-13 at 09 37 13@2x

@tzarsquared
Copy link

Chart 2, step 5: "Authenticator" doesn't need to be capitalized
Chart 2, step 10: "Browser" doesn't need to be capitalized
Chart 3, step 1: "Relaying Party" should be "relying party"
Chart 3, step 5: "Authenticator" doesn't need to be capitalized
Chart 3, step 9: "Browser" doesn't need to be capitalized

tzarsquared and others added 2 commits December 19, 2023 11:40
Updates after feedback from @tzarsquared

Chart 2, step 5: "Authenticator" doesn't need to be capitalized
Chart 2, step 10: "Browser" doesn't need to be capitalized
Chart 3, step 1: "Relaying Party" should be "relying party"
Chart 3, step 5: "Authenticator" doesn't need to be capitalized
Chart 3, step 9: "Browser" doesn't need to be capitalized
@nielthiart
Copy link
Author

Chart 2, step 5: "Authenticator" doesn't need to be capitalized Chart 2, step 10: "Browser" doesn't need to be capitalized Chart 3, step 1: "Relaying Party" should be "relying party" Chart 3, step 5: "Authenticator" doesn't need to be capitalized Chart 3, step 9: "Browser" doesn't need to be capitalized

Thanks @tzarsquared, fixed!

@RichardJECooke RichardJECooke deleted the branch master August 13, 2024 08:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants