| Summary: | [WebAuthn] Consider support for the displayName for FIDO authenticator | ||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | pascoe <pascoe> | ||||||||
| Component: | WebKit Misc. | Assignee: | pascoe <pascoe> | ||||||||
| Status: | RESOLVED FIXED | ||||||||||
| Severity: | Normal | CC: | bfulgham, ews-watchlist, jiewen_tan, webkit-bug-importer | ||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||
| Version: | WebKit Nightly Build | ||||||||||
| Hardware: | Unspecified | ||||||||||
| OS: | Unspecified | ||||||||||
| Attachments: |
|
||||||||||
|
Description
pascoe@apple.com
2021-11-19 16:28:39 PST
Created attachment 444868 [details]
Patch
Created attachment 446049 [details]
Patch
Comment on attachment 446049 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=446049&action=review > Source/WebKit/ChangeLog:9 > + Start storing the displayName and icon fields with the platform authenticator Are these part of the WebAuthn spec? If so, could you add comments in the code indicating the relevant spec step to make it easy to find/validate this later? > Source/WebKit/UIProcess/API/Cocoa/_WKWebAuthenticationPanel.mm:271 > + [credential setDictionary:@{ Could also be done with 'initWithObjectsAndKeys: ', but I doubt there's any real performance benefit to doing so. Created attachment 446381 [details]
Patch
Thank you for the review. Addressed comments and removed icon from the change, as it is a field that was removed from the webauthn spec in: https://github.com/w3c/webauthn/pull/1337 Comment on attachment 446381 [details]
Patch
r=me. Nice!
Committed r286746 (244991@main): <https://commits.webkit.org/244991@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 446381 [details]. |