Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Formalize how subsetting sending reports works #234

Open
zcorpan opened this issue Feb 25, 2021 · 2 comments
Open

Formalize how subsetting sending reports works #234

zcorpan opened this issue Feb 25, 2021 · 2 comments
Assignees

Comments

@zcorpan
Copy link
Member

zcorpan commented Feb 25, 2021

https://w3c.github.io/reporting/#send-reports

So reports, as written, is the full list of reports for context. The algorithm then iterates over all of them and "attempt to deliver" them.

Below the normative algorithm, there's a non-normative note that contains an RFC2119 keyword (not good!):

Note: User agents MAY decide to attempt delivery for only a subset of the collected reports or endpoints (because, for example, sending all the reports at once would consume an unreasonable amount of bandwidth, etc). As reports are only removed from the cache after delivery has been attempted, skipped reports will simply be delivered later.

Can this be formalized in the algorithm? For example, before step 2:

Let reportsSubset be an implementation-defined subset of reports, selecting the oldest reports and retaining relative order.

Note: This step allows user agents to attempt delivery for only a subset of the collected reports or endpoints (because, for example, sending all the reports at once would consume an unreasonable amount of bandwidth, etc). As reports are only removed from the cache after delivery has been attempted, skipped reports will simply be delivered later.

and then operate on reportsSubset instead of reports.

@clelland
Copy link
Contributor

Would "selecting the oldest reports and retaining relative order" imply that the only allowed subsets are always the oldest reports still in the queue?

@zcorpan
Copy link
Member Author

zcorpan commented Feb 25, 2021

Yeah. That's something I made up because I thought it made sense, giving it about a minute of thought. Do you agree that only allowing such subsets makes sense? Or do we want to allow any subset?

@clelland clelland self-assigned this Jun 20, 2022
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

No branches or pull requests

2 participants