Skip to content

style: prefix cached bundle helper - #58

Merged
leopoldjoy merged 2 commits into
mainfrom
audit/prefix-cached-bundle-helper
Jul 31, 2026
Merged

style: prefix cached bundle helper#58
leopoldjoy merged 2 commits into
mainfrom
audit/prefix-cached-bundle-helper

Conversation

@leopoldjoy

Copy link
Copy Markdown
Contributor

Summary

  • rename the internal certificate-bundle helper to _verifyCachedCertBundle
  • retain the existing cached-chain behavior

Audit finding

Addresses external finding #20.

Tests

  • forge fmt --check src test
  • forge test --match-path test/IndefiniteLengthCbor.t.sol

Co-authored-by: OpenCode <opencode-noreply@coinbase.com>
@leopoldjoy
leopoldjoy marked this pull request as ready for review July 27, 2026 13:41
awilliams1-cb
awilliams1-cb previously approved these changes Jul 30, 2026
…undle-helper

Co-authored-by: OpenCode <opencode-noreply@coinbase.com>

# Conflicts:
#	src/NitroValidator.sol
@leopoldjoy
leopoldjoy merged commit ed27d63 into main Jul 31, 2026
8 checks passed
@leopoldjoy
leopoldjoy deleted the audit/prefix-cached-bundle-helper branch July 31, 2026 15:02
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