Bug 240459

Summary: [WebAuthn] Set displayName in AuthenticatorAssertionResponse for platform authenticator
Product: WebKit Reporter: pascoe <pascoe>
Component: WebKit Misc.Assignee: pascoe <pascoe>
Status: RESOLVED FIXED    
Severity: Normal CC: bfulgham, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch bfulgham: review+, ews-feeder: commit-queue-

Description pascoe@apple.com 2022-05-16 09:28:45 PDT 
This prevents the display name from being passed to the UI.
Comment 1 Radar WebKit Bug Importer 2022-05-16 09:28:54 PDT 
<rdar://problem/93350479>
Comment 2 pascoe@apple.com 2022-05-16 11:36:14 PDT 
Created attachment 459440 [details]
Patch
Comment 3 pascoe@apple.com 2022-05-16 11:50:59 PDT 
Created attachment 459442 [details]
Patch
Comment 4 Brent Fulgham 2022-05-17 09:58:23 PDT 
Comment on attachment 459442 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=459442&action=review

r=me

> Source/WebKit/ChangeLog:3
> +        [WebAuthn] Set displayName in AuthenticatorAssertionResponse for platform authenticator

Question: I'm not sure how setting the displayName here "prevents the display name from being passed to the UI". Did the bug mean to say "prevents the user name from being passed to the UI?"
Comment 5 Brent Fulgham 2022-05-17 09:59:38 PDT 
The windows test failure does not seem to be related to this Cocoa and WebAuth-specific change.
Comment 6 pascoe@apple.com 2022-05-17 10:03:12 PDT 
(In reply to Brent Fulgham from comment #4)
> 
> > Source/WebKit/ChangeLog:3
> > +        [WebAuthn] Set displayName in AuthenticatorAssertionResponse for platform authenticator
> 
> Question: I'm not sure how setting the displayName here "prevents the
> display name from being passed to the UI". Did the bug mean to say "prevents
> the user name from being passed to the UI?"

Thank you for the review.

I meant to say: Not setting the displayName in AuthenticatorAssertionResponse for platform credentials is a bug. This bug prevents the UI code from seeing the displayName for a credential, if available. The patch fixes this.
Comment 7 pascoe@apple.com 2022-05-17 11:42:18 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/682
Comment 8 EWS 2022-05-17 12:03:15 PDT 
Committed r294331 (250651@main): <https://commits.webkit.org/250651@main>

Reviewed commits have been landed. Closing PR #682 and removing active labels.