Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Submission: Validate request payload #50

Closed
michael-burwig opened this issue May 8, 2020 · 1 comment · Fixed by #133
Closed

Submission: Validate request payload #50

michael-burwig opened this issue May 8, 2020 · 1 comment · Fixed by #133
Assignees
@johanneseschrig
Labels
submission service
Milestone
Alpha

Comments

@michael-burwig
Copy link
Member

michael-burwig commented May 8, 2020 •
edited
Loading

Validate the submitted temporary exposure keys wrt the latest specification, eg. expected value ranges and things like maximum number of keys per submission. Any violations shall result in an error response, HTTP 400.
@michael-burwig michael-burwig added this to the Alpha milestone May 9, 2020
@christian-kirschnick
Copy link
Member

christian-kirschnick commented May 12, 2020 •
edited
Loading

  • The diagnosis key list must contain 1 to 14 keys
  • the keyData must be valid
  • Country check -> only accept whitelisted countries (for now, hardcode to DE)
  • Check timestamp / rolling period
  • check transmission risk level

@christian-kirschnick christian-kirschnick changed the title Validate request payload Submission: Validate request payload May 15, 2020
Closed
@johanneseschrig johanneseschrig self-assigned this May 15, 2020
@michael-burwig michael-burwig linked a pull request May 17, 2020 that will close this issue
Validate Submission Payload #133
Merged
@johanneseschrig johanneseschrig mentioned this issue May 17, 2020
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@johanneseschrig johanneseschrig

Labels
submission service
Projects
None yet
Milestone
Alpha
Development

Successfully merging a pull request may close this issue.

Validate Submission Payload corona-warn-app/cwa-server
3 participants
@johanneseschrig @christian-kirschnick @michael-burwig