Bug 222076

Summary: PCM: Request signature for unlinkable token using attributionSourceNonce
Product: WebKit Reporter: John Wilander <wilander>
Component: WebKit Misc.Assignee: John Wilander <wilander>
Status: RESOLVED FIXED    
Severity: Normal CC: cdumez, changseok, esprehn+autocc, ews-watchlist, gyuyoung.kim, japhet, jiewen_tan, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Local Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 221291    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

Description John Wilander 2021-02-17 14:47:49 PST
The attributionSourceNonce from the anchor tag should be transferred to the network process and used to initiate token signing from the source server.
Comment 1 John Wilander 2021-02-17 14:48:01 PST
<rdar://73581651>
Comment 2 John Wilander 2021-02-17 15:13:40 PST
Created attachment 420731 [details]
Patch
Comment 3 John Wilander 2021-02-17 18:50:06 PST
Created attachment 420781 [details]
Patch
Comment 4 John Wilander 2021-02-17 18:50:46 PST
Comment on attachment 420781 [details]
Patch

Fixed content extensions layout test failure (renamed TestRunner function).
Comment 5 John Wilander 2021-02-17 22:25:07 PST
Created attachment 420798 [details]
Patch
Comment 6 John Wilander 2021-02-17 22:26:46 PST
Comment on attachment 420798 [details]
Patch

Fixed a bad search+replace in the failing layout test send-attribution-conversion-request.html where the URL mistakingly ended with the new signToken.php instead of the correct, pre-existing conversionReport.php.
Comment 7 John Wilander 2021-02-17 22:38:38 PST
Created attachment 420799 [details]
Patch
Comment 8 John Wilander 2021-02-17 22:39:35 PST
Comment on attachment 420799 [details]
Patch

Rebased to see if I can get it to apply cleanly.
Comment 9 John Wilander 2021-02-18 08:36:12 PST
Comment on attachment 420799 [details]
Patch

Thanks for the review, Chris!
Comment 10 EWS 2021-02-18 11:44:44 PST
Committed r273088: <https://commits.webkit.org/r273088>

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