Bug 244907 - [Reporting API] Test reports should go "Reporting-Endpoints" Endpoints
Summary: [Reporting API] Test reports should go "Reporting-Endpoints" Endpoints
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Brent Fulgham
URL:
Keywords: InRadar
Depends on:
Blocks: 189365
  Show dependency tree / graph
 
Reported: 2022-09-07 14:44 PDT by Brent Fulgham
Modified: 2022-10-21 19:33 PDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Brent Fulgham 2022-09-07 14:44:18 PDT 
Test reports should be sent to any endpoints provided in a "Report-To" header, but currently are not. There also don't appear to be WPT that check this behavior.
Comment 1 Radar WebKit Bug Importer 2022-09-07 14:44:44 PDT 
<rdar://problem/99666423>
Comment 2 Brent Fulgham 2022-10-07 14:21:50 PDT 
I should have written "Reporting-Endpoints", not "Report-To".
Comment 3 Brent Fulgham 2022-10-10 17:10:57 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/5216
Comment 4 Brent Fulgham 2022-10-12 14:13:18 PDT 
Actually, I'm wrong! There are existing tests for this behavior:

document-reporting-bypass-report-to.https.sub.html
document-reporting-named-endpoints.https.sub.html
document-reporting-override-endpoint.https.sub.html
Comment 5 Brent Fulgham 2022-10-12 14:24:22 PDT 
Wrong again! Those tests use 'deprecation' and 'document-policy-violation' reports, but don't exercise the test report feature.
Comment 6 Brent Fulgham 2022-10-17 15:32:26 PDT 
Pull request: https://github.com/Webkit/WebKit/pull/5216
Comment 7 EWS 2022-10-21 13:21:28 PDT 
Committed 255845@main (8ae6f494b997): <https://commits.webkit.org/255845@main>

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