Skip to content

feat: allow larger ws frames on client side#20307

Merged
mattsse merged 3 commits intomainfrom
matt/allow-larger-ws-frames
Dec 11, 2025
Merged

feat: allow larger ws frames on client side#20307
mattsse merged 3 commits intomainfrom
matt/allow-larger-ws-frames

Conversation

@mattsse
Copy link
Collaborator

@mattsse mattsse commented Dec 11, 2025

No description provided.

@mattsse mattsse requested a review from Rjected as a code owner December 11, 2025 14:18
@github-project-automation github-project-automation bot moved this to Backlog in Reth Tracker Dec 11, 2025
@mattsse mattsse requested a review from shekhirin December 11, 2025 14:18
@mattsse mattsse added the C-enhancement New feature or request label Dec 11, 2025
@mattsse mattsse enabled auto-merge December 11, 2025 14:19
@mattsse mattsse closed this Dec 11, 2025
auto-merge was automatically disabled December 11, 2025 14:20

Pull request was closed

@github-project-automation github-project-automation bot moved this from Backlog to Done in Reth Tracker Dec 11, 2025
@mattsse mattsse reopened this Dec 11, 2025
@github-project-automation github-project-automation bot moved this from Done to In Progress in Reth Tracker Dec 11, 2025
Copy link
Member

@shekhirin shekhirin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm one nit

ConnectionConfig::default().with_max_retries(u32::MAX).with_ws_config(
WebSocketConfig::default()
// allow larger messages/frames for big blocks
.max_frame_size(Some(128 << 20))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we have a human readable constant here

@mattsse mattsse closed this Dec 11, 2025
@mattsse mattsse reopened this Dec 11, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Dec 11, 2025
@github-project-automation github-project-automation bot moved this from Done to In Progress in Reth Tracker Dec 11, 2025
@mattsse mattsse enabled auto-merge December 11, 2025 14:55
@mattsse mattsse added this pull request to the merge queue Dec 11, 2025
Merged via the queue into main with commit 24c2981 Dec 11, 2025
99 of 121 checks passed
@mattsse mattsse deleted the matt/allow-larger-ws-frames branch December 11, 2025 16:54
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Dec 11, 2025
Vui-Chee added a commit to okx/reth that referenced this pull request Dec 12, 2025
…number

* upstream: (203 commits)
  feat(node-core): make rpc server args customizable (paradigmxyz#20312)
  feat: add `account_history_in_rocksdb` field to `StorageSettings` (paradigmxyz#20282)
  feat(engine): Add BAL stub methods to ExecutionPayload and BlockOrPayload (paradigmxyz#20311)
  docs: fix misleading links (paradigmxyz#20300)
  ci: add more sccache (paradigmxyz#20316)
  feat: bump alloy-evm (paradigmxyz#20314)
  feat: allow larger ws frames on client side (paradigmxyz#20307)
  docs: add architecture diagrams to ExEx documentation (paradigmxyz#20193)
  feat: add semaphore for blocking IO requests (paradigmxyz#20289)
  ci: scale down depot runners (paradigmxyz#20295)
  perf: fetch header directly (paradigmxyz#20294)
  docs(exex): fix DebugApi comment (paradigmxyz#20296)
  feat: add support for testing_ rpc namespace and testing_buildBlockV1 (paradigmxyz#20094)
  chore: update engine_getBlobs metric (paradigmxyz#20290)
  chore(optimism): move predeploy constant to op-alloy (paradigmxyz#20181)
  docs: fix stages order and add missing EraStage (paradigmxyz#20283)
  docs: improve map_add_ons method documentation (paradigmxyz#20248)
  feat: add `transaction_hash_numbers_in_rocksdb` field to `StorageSettings` (paradigmxyz#20209)
  docs: clarify network mode, tx gossip and NAT (paradigmxyz#20247)
  feat: add support for debug_getBadBlock (paradigmxyz#20177)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C-enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants