Skip to content

fix(batch-poster): surface AnyTrust decode errors and add Espresso ABI#95

Open
douglance wants to merge 1 commit intomainfrom
fix/anytrust-espresso-decode
Open

fix(batch-poster): surface AnyTrust decode errors and add Espresso ABI#95
douglance wants to merge 1 commit intomainfrom
fix/anytrust-espresso-decode

Conversation

@douglance
Copy link
Contributor

The AnyTrust DACert check silently swallowed decode errors for chains using non-standard batch poster functions (e.g. Espresso). This meant unknown selectors like 0x37501551 were logged to console but never reported to Slack, masking potential issues.

  • Add Espresso 7-param addSequencerL2BatchFromOrigin variant to sequencerInboxAbi so DACert verification works for Espresso chains
  • Push decode errors to the alerts array instead of only logging them, so unrecognized selectors are surfaced to Slack
  • Add tests for Espresso DACert decoding and decode-failure alerting

The AnyTrust DACert check silently swallowed decode errors for chains
using non-standard batch poster functions (e.g. Espresso). This meant
unknown selectors like 0x37501551 were logged to console but never
reported to Slack, masking potential issues.

- Add Espresso 7-param addSequencerL2BatchFromOrigin variant to
  sequencerInboxAbi so DACert verification works for Espresso chains
- Push decode errors to the alerts array instead of only logging them,
  so unrecognized selectors are surfaced to Slack
- Add tests for Espresso DACert decoding and decode-failure alerting
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