-
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
Restore AES-GCM as the cipher mode across all implementations #16
Milestone
Comments
gnarea
added
bug
Something isn't working
libs
Applicable to the JS/JVM/etc libraries
labels
Nov 25, 2020
gnarea
added a commit
to relaycorp/relaynet-core-js
that referenced
this issue
Nov 25, 2020
gnarea
added a commit
to relaycorp/awala-jvm
that referenced
this issue
Nov 25, 2020
github-actions bot
pushed a commit
to relaycorp/relaynet-core-js
that referenced
this issue
Nov 25, 2020
gnarea
added a commit
to relaycorp/relaynet-gateway-android
that referenced
this issue
Nov 25, 2020
See: relaycorp/relayverse#16 The counterpart to this change will land on the server in a few minutes: relaycorp/awala-gateway-internet#299
kodiakhq bot
pushed a commit
to relaycorp/relaynet-gateway-android
that referenced
this issue
Nov 25, 2020
This change will allow the Android GW to (de)encrypt cargo exchanged with the public gateway. See: relaycorp/relayverse#16 The counterpart to this change will land on the server in a few minutes: relaycorp/awala-gateway-internet#299
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As a workaround for PeculiarVentures/PKI.js#287, I had to downgrade the cipher mode from GCM to CBC across the board. This is OK for the current phase of Relaynet, but the lack of authenticated encryption is going to block the eventual General Availability of Relaynet.
Note that the lack of support for AES-GCM is a violation of RS-018: https://specs.relaynet.network/RS-018#symmetric-ciphers
The eventual fix should reinstate support for AES-GCM and make it the default, whilst still supporting AES-CBC for backwards compatibility.
See also:
The text was updated successfully, but these errors were encountered: