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

Remove note hash's nullifier counter. #7250

Closed
LeilaWang opened this issue Jul 1, 2024 · 1 comment
Closed

Remove note hash's nullifier counter. #7250

LeilaWang opened this issue Jul 1, 2024 · 1 comment

Comments

@LeilaWang
Copy link
Collaborator

Problem

Currently we assign a nullifier counter to a pending note hash. If this value is not zero, it means the note hash is nullified in the same tx by the nullifier with that specific counter.

We use this counter to check that, when reading a pending note, the read request is emitted before the note hash's nullified.

Although this check makes sense, it doesn't align with how reading a settled note hash works. A read request for a settled note hash is valid as long as the membership check succeeds, even if its nullifier has been emitted.

Solution

Removing this check because this shouldn't be enforced by the protocol only for reading pending note hashes. The developers should be able to write the same code and expect the same outcome for reading either pending or settled note hashes.

@LeilaWang
Copy link
Collaborator Author

Closed by #7294

@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

No branches or pull requests

1 participant