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.

Sign up for GitHub

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

Jump to bottom

feat(Keys_Viewing): Compute ciphertext header #5867

Closed
Tracked by #5606
LHerskind opened this issue Apr 19, 2024 · 0 comments · Fixed by #6175
Closed
Tracked by #5606

feat(Keys_Viewing): Compute ciphertext header #5867

LHerskind opened this issue Apr 19, 2024 · 0 comments · Fixed by #6175
Assignees
@LHerskind

Comments

@LHerskind
Copy link
Contributor

LHerskind commented Apr 19, 2024 •
edited
Loading

For AES in noir: preferably use #5866, otherwise use #5895 if required.

Implementation in noir.

From #5896 should be functionality to compute the symmetric key.

We need to compute the ciphertext header, which currently seems to just be the contract_address for the note.

As with many of the others, better description is in the diagam linked at the main issue.

As the flows for both incoming and outgoing is the same, simply with different pub keys, it seems fine to just have one function implementation.

image
@LHerskind LHerskind mentioned this issue Apr 19, 2024
Closed
@github-project-automation github-project-automation bot moved this to Todo in A3 Apr 19, 2024
This was referenced Apr 19, 2024
Closed
Closed
@LHerskind LHerskind assigned LHerskind and unassigned nventuro May 3, 2024
@LHerskind LHerskind mentioned this issue May 3, 2024
Merged
LHerskind added a commit that referenced this issue May 7, 2024
@LHerskind
feat: add ciphertext computation for log header (#6175)
3e05534 
Fixes #5867 with the AES oracle on the noir side and a encrypt/decrypt
tool on the typescript side as well.

Changes the symmetric key derivation to use `GrumpkinPrivateKey` instead
of the `GrumpkinScalar`, this changes the order of low/high.
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 May 7, 2024
This was referenced May 8, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LHerskind LHerskind

Labels
None yet
Projects
A3
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add ciphertext computation for log header AztecProtocol/aztec-packages
2 participants
@nventuro @LHerskind