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

Repeated requests PUT /tasks/{tasks}/aggregation_jobs/{aggregation-job-id} are handled as fresh aggregation jobs #522

Open
cjpatton opened this issue Mar 4, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@cjpatton
Copy link
Contributor

cjpatton commented Mar 4, 2024

DAP wants to allow the Leader to repeat requests to:

PUT /tasks/{tasks}/aggregation_jobs/{aggregation-job-id}

As long as the body of the request is the same, we should respond with the same AggregationJobRep as we did the first time the aggregation job was initialized.

At the moment, we don't keep track of past aggregation jobs: we would handle the repeated request as if it were fresh and respond by rejecting each of the reports.

@cjpatton cjpatton added the bug Something isn't working label Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant