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

eax: allow variable length nonces #239

Open
lopsided98 opened this issue Nov 28, 2020 · 4 comments
Open

eax: allow variable length nonces #239

lopsided98 opened this issue Nov 28, 2020 · 4 comments

Comments

@lopsided98
Copy link

Similar to #62, except with EAX. I would like to be able to interoperate with an existing protocol that uses 32-bit nonces. Presumably this could be implemented similarly to #126, but I'm not familiar enough with the codebase to do it easily.

@Flakebi
Copy link
Contributor

Flakebi commented Jan 9, 2021

Is what you need implemented in #231? :)

@tarcieri
Copy link
Member

I think so. Please reopen if not.

@lopsided98
Copy link
Author

No, that PR doesn't implement what I need. That allows variable length tags, whereas I need variable length nonces. The nonce size is still hardcoded to the block size: https://github.com/RustCrypto/AEADs/blob/master/eax/src/lib.rs#L192

@tarcieri tarcieri reopened this Jan 15, 2021
lopsided98 added a commit to lopsided98/AEADs that referenced this issue Apr 16, 2021
lopsided98 added a commit to lopsided98/AEADs that referenced this issue May 7, 2021
lopsided98 added a commit to lopsided98/AEADs that referenced this issue May 8, 2021
lopsided98 added a commit to lopsided98/AEADs that referenced this issue Dec 22, 2022
@tracktwo
Copy link

I have the same issue with a different protocol: eddystone ETLM uses 48-bit nonce (https://github.com/google/eddystone/blob/master/eddystone-tlm/tlm-encrypted.md). I think the solution used for aesgcm would be enough here too, although it's unfortunate that the order of the generic parameters for nonce and tag sizes would not match the other algorithm without a breaking change.

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

5 participants
@tarcieri @lopsided98 @Flakebi @tracktwo and others