Skip to content

l2geth: bring back unsupported RPC methods#2007

Merged
tynes merged 2 commits intodevelopfrom
feat/bring-back-unsupported-rpcs
Feb 2, 2022
Merged

l2geth: bring back unsupported RPC methods#2007
tynes merged 2 commits intodevelopfrom
feat/bring-back-unsupported-rpcs

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented Jan 13, 2022

l2geth: bring back unsupported RPC methods

Previously, any RPC methods involving hot keys and
signing by the node were disabled when the node
ran with the OVM configured. This was to prevent
users from attempting to use these methods against
the live sequencer. Now that the infrastructure is
more mature with proxyd, particular RPC requests
can be routed appropriately and blocked at the infra
level. Allowing these methods makes local development
easier.

This change cannot be adopted without coordination
from infra providers, ensuring that they will block
these methods at their infrastructure. Each optimism
node has at least 1 key that is used to ensure
block production is deterministic when running in
clique mode.

Description
A clear and concise description of the features you're adding in this pull request.

Additional context
Add any other context about the problem you're solving.

Metadata

  • Fixes #[Link to Issue]

@changeset-bot
Copy link

changeset-bot bot commented Jan 13, 2022

🦋 Changeset detected

Latest commit: 0bc5365

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/l2geth Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@codecov-commenter
Copy link

codecov-commenter commented Jan 13, 2022

Codecov Report

Merging #2007 (81d9056) into develop (f8d62ac) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2007   +/-   ##
========================================
  Coverage    75.37%   75.37%           
========================================
  Files           81       81           
  Lines         2705     2705           
  Branches       436      436           
========================================
  Hits          2039     2039           
  Misses         666      666           
Flag Coverage Δ
batch-submitter 62.63% <ø> (ø)
contracts 90.48% <ø> (ø)
core-utils 59.94% <ø> (ø)
data-transport-layer 38.64% <ø> (ø)
message-relayer 70.86% <ø> (ø)
sdk 86.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f8d62ac...81d9056. Read the comment docs.

@tynes
Copy link
Contributor Author

tynes commented Jan 18, 2022

This is blocked being used in production until proxyd is deployed on the edge for mainnet with configuration for not allowing the RPC methods that allow for key usage. eth_sign, eth_sendTransaction, eth_signTransaction or Quicknode explicitly blocks these methods. Infura/Alchemy do block these methods, calling out to Quicknode now seems to allow these methods through

@tynes tynes force-pushed the feat/bring-back-unsupported-rpcs branch from 46eb225 to e23be8d Compare January 21, 2022 18:58
@tynes
Copy link
Contributor Author

tynes commented Jan 24, 2022

This has a merge conflict now that needs to be fixed

l2geth: bring back unsupported RPC methods

Previously, any RPC methods involving hot keys and
signing by the node were disabled when the node
ran with the OVM configured. This was to prevent
users from attempting to use these methods against
the live sequencer. Now that the infrastructure is
more mature with `proxyd`, particular RPC requests
can be routed appropriately and blocked at the infra
level. Allowing these methods makes local development
easier.

This change cannot be adopted without coordination
from infra providers, ensuring that they will block
these methods at their infrastructure. Each optimism
node has at least 1 key that is used to ensure
block production is deterministic when running in
clique mode.

Also forward transactions to the sequencer when
running as the verifier for the RPC endpoint
`eth_sendTransaction`. This RPC utilizes a key
that the node is managing.
@tynes tynes force-pushed the feat/bring-back-unsupported-rpcs branch from e23be8d to 81d9056 Compare January 31, 2022 19:54
@tynes tynes merged commit f3989c0 into develop Feb 2, 2022
@tynes tynes deleted the feat/bring-back-unsupported-rpcs branch February 2, 2022 02:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-cannon Area: cannon