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.

Sign up for GitHub

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

Jump to bottom

Remove mutable information from credential ID generation #1458

Closed
nsatragno opened this issue Jul 22, 2020 · 0 comments · Fixed by #1460
Closed

Remove mutable information from credential ID generation #1458

nsatragno opened this issue Jul 22, 2020 · 0 comments · Fixed by #1460
Assignees
@nsatragno
Labels
type:editorial
Milestone
L2-WD-04 Final

Comments

@nsatragno
Copy link
Member

nsatragno commented Jul 22, 2020 •
edited
Loading

(this is branched from #1456)

We should update generation of credential IDs that wrap the public key credential source to remove mutable fields from it (at the moment anything under otherUI such as displayName, name). Credential IDs are immutable, and afaict those fields aren't even used anywhere at the moment.
@nsatragno nsatragno added this to the L2-WD-04 Final milestone Jul 22, 2020
@nsatragno nsatragno self-assigned this Jul 22, 2020
@nsatragno nsatragno changed the title Remove mutable information from stateless credential ID generation Remove mutable information from credential ID generation Jul 22, 2020
@nsatragno nsatragno mentioned this issue Jul 29, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nsatragno nsatragno

Labels
type:editorial
Projects
None yet
Milestone
L2-WD-04 Final
Development

Successfully merging a pull request may close this issue.

Do not wrap otherUI into credential ID nsatragno/webauthn
2 participants
@nsatragno @nadalin