Skip to content

Conversation

@ThomasQuesadilla
Copy link
Collaborator

@ThomasQuesadilla ThomasQuesadilla commented Mar 12, 2025

Adds language to allow user-agents to delete reports before delivery based on implementation-defined criteria.


Preview | Diff

@apasel422 apasel422 requested a review from linnan-github March 12, 2025 18:04
@linnan-github linnan-github requested a review from arpanah March 13, 2025 19:03
@apasel422 apasel422 self-requested a review March 17, 2025 12:27
@ThomasQuesadilla ThomasQuesadilla changed the title Explicit Report Lifetime When Persisted Introduce Report Data Lifetime Mar 17, 2025
@ThomasQuesadilla ThomasQuesadilla force-pushed the tquintanilla-report-ttl branch 3 times, most recently from 0adf153 to 918b92a Compare March 19, 2025 17:30
@linnan-github
Copy link
Collaborator

Limit report data lifetime to align with source data lifetime. Introduces a vendor-specific value to control the explicit lifetime value,

Preview | Diff

Please update the description as no vendor-specific value is introduced.

@ThomasQuesadilla ThomasQuesadilla merged commit 57c2811 into main Apr 1, 2025
2 checks passed
@ThomasQuesadilla ThomasQuesadilla deleted the tquintanilla-report-ttl branch April 1, 2025 19:33
aarongable pushed a commit to chromium/chromium that referenced this pull request Apr 7, 2025
WICG/attribution-reporting-api#1505

The 30 day limit starts from the report's initial report time. We delete
expired reports right before they would otherwise be sent.

Bug: 402791062
Change-Id: I91da5a00052cdce42e8f69267da5b6f0b204543c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6207729
Commit-Queue: Thomas Quintanilla <[email protected]>
Reviewed-by: Charlie Harrison <[email protected]>
Reviewed-by: Andrew Paseltiner <[email protected]>
Reviewed-by: Dominic Farolino <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1443528}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants