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

Spec needs to mention how AAD is formatted. #6

Closed
insanum opened this issue Sep 28, 2022 · 3 comments
Closed

Spec needs to mention how AAD is formatted. #6

insanum opened this issue Sep 28, 2022 · 3 comments

Comments

@insanum
Copy link

insanum commented Sep 28, 2022

I didn't see any mention of the AAD format in the spec. I think it should be added.
Looking at the code I see the AAD is the entire PSP header plus any non-encrypted bytes (i.e., based on the crypt offset).

@hlrichardson
Copy link
Collaborator

The document also has a diagram showing that authenticated data includes all bytes from the beginning of the PSP header through the last payload byte, and encrypted data includes all bytes from crypt offset through the last payload byte. Would adding some text to say that the encryption method is AEAD where the associated data is from the first byte of the PSP header up to the crypt offset help clarify this?

@insanum
Copy link
Author

insanum commented Sep 28, 2022

Hey Lance. :-) Yes, what you describe should suffice.

@hlrichardson
Copy link
Collaborator

Spec has been updated as suggested. Thanks for the feedback!

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

2 participants