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

[coepi-backend-aws][v4]: Do report signature validation on the server side before persisting the data #9

Closed
ramnanib2 opened this issue Apr 13, 2020 · 3 comments
Assignees

Comments

@ramnanib2
Copy link
Collaborator

ramnanib2 commented Apr 13, 2020

Currently, the v4 API endpoint receives the report data as a POST request, stores the report data buffer as is and serves it in a GET request. It does not perform any report validation and signature verification as per the TCN protocol. There is a Kotlin library that can be used to achieve that. Report validation and signature verification need to be added here -> https://github.com/Co-Epi/coepi-backend-aws/blob/master/src/main/kotlin/org/coepi/api/v4/TCNCloudAPIHandler.kt#L141-L142

This requires knowledge of the TCN protocol:
https://github.com/TCNCoalition/TCN/blob/main/README.md

There is already a Kotlin library that can be re-used:
https://github.com/TCNCoalition/tcn-client-android/tree/develop/tcn-client-android/src/main/java/org/tcncoalition/tcnclient/crypto

@ramnanib2 ramnanib2 changed the title [coepi-backend-aws]: Do report signature validation on the server side before persisting the data [coepi-backend-aws][v4]: Do report signature validation on the server side before persisting the data Apr 13, 2020
@ramnanib2
Copy link
Collaborator Author

ramnanib2 commented Apr 14, 2020

This library can probably be shared with the android app. Make sure to check if it's already been developed or under development.

@ramnanib2
Copy link
Collaborator Author

Completed with 4fb51b6

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

1 participant