Skip to content

Commit

Permalink
Resolve todo on supporting legacy conversion pings
Browse files Browse the repository at this point in the history
Removes the attributionsrc requirement for trigger registration, fixing #347
  • Loading branch information
johnivdel authored Mar 14, 2022
1 parent d732ca5 commit c758d54
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions EVENT.md
Original file line number Diff line number Diff line change
Expand Up @@ -253,8 +253,11 @@ or JavaScript:
window.attributionReporting.registerAttributionTrigger(
"https://adtech.example/attribution_trigger?purchase=13")
```
TODO: Finalize whether the `attributionsrc` attribute should be optional to
support legacy HTML tags for triggers.

As a stop-gap to support pre-existing conversion tags which do not include the
`attributionsrc` attribute, or use a different Fetch API, the browser will also process
trigger registration headers for all subresource requests on the page where
the `attribution-reporting` Permissions Policy is enabled.

Like source event registrations, these requests should respond with a new HTTP
header `Attribution-Reporting-Register-Event-Trigger` which contains information
Expand Down

0 comments on commit c758d54

Please sign in to comment.