-
Notifications
You must be signed in to change notification settings - Fork 0
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
Cargo Delivery Authorisation issuer isn't rotated #571
Comments
gnarea
changed the title
Cargo Delivery Authorisation certificate isn't rotated
Cargo Delivery Authorisation issuer isn't rotated
Feb 25, 2022
gnarea
added a commit
to relaycorp/relaynet-core-js
that referenced
this issue
Feb 26, 2022
To avoid making this mistake in the JS implementation: relaycorp/relaynet-gateway-android#571
kodiakhq bot
pushed a commit
to relaycorp/relaynet-core-js
that referenced
this issue
Feb 26, 2022
To avoid making this mistake in the JS implementation: relaycorp/relaynet-gateway-android#571
Tasks:
|
🎉 This issue has been resolved in version 1.6.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I forgot to call this out explicitly in the issues: We should rotate the self-issued certificate used in Cargo Delivery Authorisations.
Right now, we can only have exactly one as it's stored in a file that's generated the first time the app starts (and there's no rotation):
relaynet-gateway-android/app/src/main/java/tech/relaycorp/gateway/domain/LocalConfig.kt
Line 116 in 809ebbe
Instead, we should do what we've been doing with the other certificates:
relaynet-gateway-android/app/src/main/java/tech/relaycorp/gateway/domain/courier/GenerateCCA.kt
Lines 27 to 32 in 8f01a2b
relaynet-gateway-android/app/src/main/java/tech/relaycorp/gateway/data/disk/CargoStorage.kt
Line 30 in c56cc71
We probably want to reuse our
FileCertificateStore
and introduce a second instance to manage CDA issuers only. I think that'd require changing theFileCertificateStore
class to take a path prefix (under the existing root), so that we avoid overriding files.The text was updated successfully, but these errors were encountered: