Skip to content

feat: add ETHEREUM_HTTP_TIMEOUT_MS env var for viem HTTP transport (backport #20919)#21434

Merged
spalladino merged 3 commits intobackport-to-v4-next-stagingfrom
claudebox/02b67d20aae96377-2
Mar 12, 2026
Merged

feat: add ETHEREUM_HTTP_TIMEOUT_MS env var for viem HTTP transport (backport #20919)#21434
spalladino merged 3 commits intobackport-to-v4-next-stagingfrom
claudebox/02b67d20aae96377-2

Conversation

@AztecBot
Copy link
Collaborator

Summary

Backport of #20919 to v4-next.

Adds ETHEREUM_HTTP_TIMEOUT_MS env var to configure the timeout for viem HTTP transports used in L1 RPC calls. Introduces makeL1HttpTransport helper to centralize transport creation with consistent timeout support.

Conflict Resolution

Cherry-pick had a conflict in blob-client/src/client/http.ts because next had refactored getSlotNumber to accept parentBeaconBlockRoot as a parameter. Resolved by keeping the v4-next code structure and applying only the transport change (makeL1HttpTransport with timeout support).

Commits

  1. Cherry-pick with conflict markers (for reviewability)
  2. Conflict resolution

ClaudeBox log: https://claudebox.work/s/02b67d20aae96377?run=2

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 12, 2026
@ludamad ludamad force-pushed the backport-to-v4-next-staging branch from 5c50c04 to 4eb8703 Compare March 12, 2026 19:33
@ludamad ludamad force-pushed the claudebox/02b67d20aae96377-2 branch from 0a5fe75 to c292133 Compare March 12, 2026 19:40
@spalladino spalladino marked this pull request as ready for review March 12, 2026 20:13
@spalladino spalladino enabled auto-merge (squash) March 12, 2026 20:13
@AztecBot AztecBot force-pushed the claudebox/02b67d20aae96377-2 branch from c292133 to 0a5fe75 Compare March 12, 2026 20:25
@ludamad
Copy link
Collaborator

ludamad commented Mar 12, 2026

Whoops, sorry, meant to hit update on a different PR.

@AztecBot AztecBot force-pushed the claudebox/02b67d20aae96377-2 branch from 6afeb7b to 73d484d Compare March 12, 2026 21:39
@AztecBot
Copy link
Collaborator Author

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/4432eb3c309f86d5�4432eb3c309f86d58;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/reqresp/reqresp_no_handshake.test.ts (236s) (code: 0) group:e2e-p2p-epoch-flakes

@spalladino spalladino merged commit f2fc318 into backport-to-v4-next-staging Mar 12, 2026
9 checks passed
@spalladino spalladino deleted the claudebox/02b67d20aae96377-2 branch March 12, 2026 22:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants