Bug 223547 - Provide a way to specify platform vs security key when requesting an assertion in the SPI
Summary: Provide a way to specify platform vs security key when requesting an assertio...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jiewen Tan
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-03-19 17:56 PDT by Jiewen Tan
Modified: 2021-03-23 13:00 PDT (History)
11 users (show)

See Also:


Attachments
Patch (50.81 KB, patch)
2021-03-19 18:46 PDT, Jiewen Tan
no flags Details | Formatted Diff | Diff
Patch (41.56 KB, patch)
2021-03-22 17:01 PDT, Jiewen Tan
no flags Details | Formatted Diff | Diff
Patch (42.23 KB, patch)
2021-03-23 01:02 PDT, Jiewen Tan
bfulgham: review+
bfulgham: commit-queue-
Details | Formatted Diff | Diff
Patch for landing (41.99 KB, patch)
2021-03-23 12:11 PDT, Jiewen Tan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jiewen Tan 2021-03-19 17:56:17 PDT
Provide a way to specify platform vs security key when requesting an assertion in the SPI.
Comment 1 Jiewen Tan 2021-03-19 17:56:34 PDT
rdar://74034746
Comment 2 Jiewen Tan 2021-03-19 18:46:10 PDT
Created attachment 423799 [details]
Patch
Comment 3 Jiewen Tan 2021-03-22 17:01:01 PDT
Created attachment 423962 [details]
Patch
Comment 4 Jiewen Tan 2021-03-23 01:02:41 PDT
Created attachment 423994 [details]
Patch
Comment 5 Brent Fulgham 2021-03-23 10:38:10 PDT
Comment on attachment 423994 [details]
Patch

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

r=me

> Source/WebCore/ChangeLog:5
> +        rdar://74034746

nit: <rdar://74034746>

> Source/WebCore/ChangeLog:15
> +        * Modules/webauthn/AuthenticatorAttachment.idl: Copied from Source/WebKit/UIProcess/API/Cocoa/_WKPublicKeyCredentialRequestOptions.mm.

I would say these two were "Moved from ...."

> Source/WebKit/ChangeLog:5
> +        rdar://74034746

Ditto

> Source/WebKit/ChangeLog:12
> +        * UIProcess/API/Cocoa/_WKAuthenticatorAttachment.h: Copied from Source/WebKit/UIProcess/API/Cocoa/_WKPublicKeyCredentialRequestOptions.mm.

Ditto "Moved from ..."
Comment 6 Jiewen Tan 2021-03-23 12:08:30 PDT
Comment on attachment 423994 [details]
Patch

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

Thanks Brent for r+ this patch.

>> Source/WebCore/ChangeLog:5
>> +        rdar://74034746
> 
> nit: <rdar://74034746>

Fixed.

>> Source/WebCore/ChangeLog:15
>> +        * Modules/webauthn/AuthenticatorAttachment.idl: Copied from Source/WebKit/UIProcess/API/Cocoa/_WKPublicKeyCredentialRequestOptions.mm.
> 
> I would say these two were "Moved from ...."

It was auto-generated. Fixed.

>> Source/WebKit/ChangeLog:5
>> +        rdar://74034746
> 
> Ditto

Fixed.

>> Source/WebKit/ChangeLog:12
>> +        * UIProcess/API/Cocoa/_WKAuthenticatorAttachment.h: Copied from Source/WebKit/UIProcess/API/Cocoa/_WKPublicKeyCredentialRequestOptions.mm.
> 
> Ditto "Moved from ..."

Fixed.
Comment 7 Jiewen Tan 2021-03-23 12:11:33 PDT
Created attachment 424047 [details]
Patch for landing
Comment 8 EWS 2021-03-23 13:00:56 PDT
Committed r274894: <https://commits.webkit.org/r274894>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 424047 [details].