-
Notifications
You must be signed in to change notification settings - Fork 242
[Do not merge yet] Add requestUserInfo
#2358
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
base: main
Are you sure you want to change the base?
Conversation
984e300 to
ddffc4d
Compare
This patch adds `requestUserInfo` to `PublicKeyCredentialUserEntity`. This allows a relying party to request user identifiers and attributes alongside a WebAuthn credential on `create()` to aid with account creation. Fixes 2336.
ddffc4d to
3001dc4
Compare
|
This is ready for an initial pass, although I expect it needs more work. |
requestUserInforequestUserInfo
|
|
||
| : <dfn>\[[userInfo]]</dfn> | ||
| :: This [=internal slot=] contains the [=user information=] requested by the [=[RP]=]. | ||
| If [=user information=] was not requested, contains the value is not present. |
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.
contains no value or is null? or is empty.
|
(Converted to draft to reinforce the "[Do not merge yet]" part.) |
|
It wouldn't let me tag people for review as a draft I think |
|
Oh, really? Ok, sorry, feel free to change it back if needed. |
This patch adds
requestUserInfotoPublicKeyCredentialUserEntity. This allows a relying party to request user identifiers and attributes alongside a WebAuthn credential oncreate()to aid with account creation.Closes #2336
The following tasks have been completed:
Implementation commitment:
Documentation and checks
Preview | Diff