Skip to content

feat(node-core): make rpc server args customizable#20312

Merged
mattsse merged 7 commits intoparadigmxyz:mainfrom
figtracer:make-rpc-server-args-customizable
Dec 11, 2025
Merged

feat(node-core): make rpc server args customizable#20312
mattsse merged 7 commits intoparadigmxyz:mainfrom
figtracer:make-rpc-server-args-customizable

Conversation

@figtracer
Copy link
Contributor

adds #20305

@figtracer figtracer marked this pull request as ready for review December 11, 2025 15:35
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 overall, can we also do the same for optional arguments? See #20142 for reference

@shekhirin shekhirin added C-enhancement New feature or request A-cli Related to the reth CLI labels Dec 11, 2025
@figtracer
Copy link
Contributor Author

LGTM overall, can we also do the same for optional arguments? See #20142 for reference

on it

/// This is __not__ used for the authenticated engine-API RPC server, see
/// `--authrpc.jwtsecret`.
#[arg(long = "rpc.jwtsecret", value_name = "HEX", global = true, required = false)]
#[arg(long = "rpc.jwtsecret", value_name = "HEX", global = true, required = false, default_value = Resettable::from(DefaultRpcServerArgs::get_global().rpc_jwtsecret.as_ref().map(|v| format!("{:?}", v).into())))]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

idk if using format! as a workaround can cause formatting problems

@figtracer figtracer requested a review from shekhirin December 11, 2025 18:36
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, just the lint needs fixing

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Dec 11, 2025
@mattsse mattsse added this pull request to the merge queue Dec 11, 2025
Merged via the queue into paradigmxyz:main with commit ed3a8a0 Dec 11, 2025
45 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Dec 11, 2025
@figtracer figtracer deleted the make-rpc-server-args-customizable branch December 11, 2025 23:47
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

A-cli Related to the reth CLI C-enhancement New feature or request

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants