Skip to content

perf(rpc): Replace magic error code number to INTERNAL_ERROR_CODE#125

Merged
louisliu2048 merged 1 commit intomainfrom
jimmyshi/32603
Jan 28, 2026
Merged

perf(rpc): Replace magic error code number to INTERNAL_ERROR_CODE#125
louisliu2048 merged 1 commit intomainfrom
jimmyshi/32603

Conversation

@JimmyShi22
Copy link
Contributor

@JimmyShi22 JimmyShi22 commented Jan 27, 2026

This pull request makes a minor improvement to error handling in the crates/rpc/src/xlayer_ext.rs file by replacing a hardcoded error code with a named constant for better readability and maintainability.

  • Error Handling Improvement:
    • Replaced the hardcoded JSON-RPC error code -32603 with the INTERNAL_ERROR_CODE constant from jsonrpsee::types::error for consistency and clarity in error responses. [1] [2]

@louisliu2048 louisliu2048 merged commit a3e4168 into main Jan 28, 2026
Vui-Chee added a commit that referenced this pull request Jan 30, 2026
* dev:
  fix: upgrade reth v1.10.2 version with bug fixes (#134)
  Update dependencies, use reth v1.10.2 and builder v0.3.0 (#133)
  chore: fix rust formatter (#132)
  chore: better rename, fix unused variable warning (#99) (#131)
  Fix comments (#127)
  Add trace log on tx cache hits (#126)
  replace magic error code number to INTERNAL_ERROR_CODE (#125)
  feat(legacy): remove eth_transactionPreExec (#124)
  fix tools readme (#123)
  fix(rpc): improve parameter validation in get_logs and service methods (#121)
  remove output file on error (#119)
  feat(legacy): support legacy routing for batch requests (#117)
  fix(legacy): avoid possible panic on malformed block params (#111)
  fallback to legacy if res is empty (#118)
  refactor(rpc): remove transaction_pre_exec API (#120)
  use XLayerPayloadServiceBuilder to simplify main() codes, merge to main branch (#112)
  fix(tools): check end block >= start block (#116)
  forward req to legacy on err (#110)
  fix: resolve audit issues related to flashblocks subscription (#113)
  fix(rpc): add trace_address for delegatecall and fix address format c… (#104)
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.

2 participants