Skip to content

op-batcher: enforce Espresso network timeouts at one seam instead of per call site #501

Description

@philippecamacho

Enforce timeouts at one seam instead of per call site. The submitter uses 3 of the SDK's 14 methods (SubmitTransaction, FetchTransactionByHash, FetchLatestBlockHeight); a thin wrapper applying NetworkTimeout inside those, plus the same bound inside batcherL1Adapter/batcherL2Adapter (which the streamer's reads already go through, so Peek gets covered too), fixes the whole unbounded-context class from my review and means a future call site can't forget the timeout. The ad-hoc networkTimeoutCtx blocks at individual call sites can then go.

Original discussion: #459 (comment)

Activity

  1. jjeangal commented on Sep 9, 2026

    @jjeangal
    Collaborator

    Largely already done. #500 adds boundedEspressoClient (the NetworkTimeout wrapper around the 3 used SDK methods) and ethereum-optimism#4855 bounds the SDK client + splits the deadline per node. Don't reimplement. Only leftover is a small cleanup on top of #500: move the Peek bound into batcherL1Adapter/batcherL2Adapter and drop the ad-hoc networkTimeoutCtx.

  2. philippecamacho commented on Sep 30, 2026

    @philippecamacho
    CollaboratorAuthor

    PR: #534

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions