Skip to content

Commit

Permalink
Clarify rejection criterion for future reports
Browse files Browse the repository at this point in the history
The spec encourages rejection of reports with timestamps too far in the
future. The Aggregator's action in this case is unspecified. It says if
a report is rejected because it's too searly, then the Aggregator
"SHOULD mark an input share rejected with error `report_too_early`".
This allows for the possibility of a different error.
  • Loading branch information
cjpatton committed Jan 14, 2025
1 parent f4d7335 commit 45a8577
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions draft-ietf-ppm-dap.md
Original file line number Diff line number Diff line change
Expand Up @@ -1876,10 +1876,11 @@ following checks:
1. Check that the input share can be decoded as specified by the VDAF. If not,
the input share MUST be marked as invalid with the error `invalid_message`.

1. Check if the report is too far into the future. Implementors can provide for
some small leeway, usually no more than a few minutes, to account for clock
skew. If a report is rejected for this reason, the Aggregator SHOULD mark the
input share as invalid with the error `report_too_early`.
1. Check if the report's timestamp is too far in the future. If the report's
timestamp is more than a few minutes ahead of the current time, then the
Aggregator SHOULD mark the input share as invalid with error
`report_too_early`. Implementations may provide for some small leeway to
account for clock skew.

1. Check if the report's timestamp is before the task's `task_start` time. If
so, the Aggregator MUST mark the input share as invalid with the error
Expand Down

0 comments on commit 45a8577

Please sign in to comment.