Skip to content

fix: configure initial certificate roles#62

Draft
leopoldjoy wants to merge 1 commit into
mainfrom
audit/configure-initial-admin-roles
Draft

fix: configure initial certificate roles#62
leopoldjoy wants to merge 1 commit into
mainfrom
audit/configure-initial-admin-roles

Conversation

@leopoldjoy

Copy link
Copy Markdown
Contributor

Summary

  • accept initial owner and revoker addresses in the CertManager constructor
  • reject zero role addresses and emit initialization events for the configured roles
  • update deployment docs, changelog, tests, and demo/test deployments

Audit finding

Addresses external finding #18.

Tests

  • forge fmt --check src test script
  • forge test (205 passed, 1 skipped)
  • production CertManager runtime remains within EIP-170

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