Skip to content

fix: emit certificate events on state transitions#61

Draft
leopoldjoy wants to merge 1 commit into
mainfrom
audit/emit-cert-state-transitions
Draft

fix: emit certificate events on state transitions#61
leopoldjoy wants to merge 1 commit into
mainfrom
audit/emit-cert-state-transitions

Conversation

@leopoldjoy

Copy link
Copy Markdown
Contributor

Summary

  • emit certificate revocation events only when the stored status changes
  • retain idempotent authorized revoke and unrevoke calls
  • make the demo helper private immutable to preserve EIP-170 deployability after the production change

Audit finding

Addresses external finding #9.

Tests

  • forge fmt --check src test
  • forge test --match-contract CertRevocationEventTest
  • forge build --sizes --skip test --skip script

Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
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.

1 participant