Skip to content

fix: remove one-time nullifier block in v4 uniqueness verify#1862

Merged
Takaros999 merged 1 commit intomainfrom
takis/remove-one-time-block
Apr 14, 2026
Merged

fix: remove one-time nullifier block in v4 uniqueness verify#1862
Takaros999 merged 1 commit intomainfrom
takis/remove-one-time-block

Conversation

@Takaros999
Copy link
Copy Markdown
Contributor

Summary

  • Removes the one-time verification restriction for v4 uniqueness proofs in production
  • Nullifier reuse is now always allowed — proofs can be verified multiple times
  • Supports auditable log workflows and HITL use cases that need to re-verify the same proof

What changed

  • Removed the allowNullifierReuse conditional in web/api/v4/verify/uniqueness-proof/handler.ts
  • Existing nullifier now always returns 200 success instead of 400 max_verifications_reached
  • Race condition on insert also returns 200 success unconditionally

Test plan

  • Verify a v4 uniqueness proof in production environment
  • Re-verify the same proof — should return 200 with nullifier_reused: true
  • Verify staging and v3.0 paths still work as before

🤖 Generated with Claude Code

Allow nullifier reuse in v4 production so proofs can be verified
multiple times, supporting auditable logs and HITL workflows.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@Gr1dlock
Copy link
Copy Markdown
Contributor

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c569fa37c4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Takaros999 Takaros999 merged commit c0b6462 into main Apr 14, 2026
13 checks passed
@Takaros999 Takaros999 deleted the takis/remove-one-time-block branch April 14, 2026 13:09
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

Successfully merging this pull request may close these issues.

2 participants