feat: Integrate op-node with op-signer for block payload signing #59
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mininny
force-pushed
the
feature/mininny/integrate-op-signer-op-node
branch
from
28e1d8c to
ea91644
Compare
zhwrd
approved these changes
Oct 8, 2024
mininny
force-pushed
the
feature/mininny/integrate-op-signer-op-node
branch
from
4dcaafb to
aad65af
Compare
mininny
force-pushed
the
feature/mininny/integrate-op-signer-op-node
branch
3 times, most recently
from
250005b to
1f5c0b7
Compare
mininny
force-pushed
the
feature/mininny/integrate-op-signer-op-node
branch
from
1f5c0b7 to
7f5e370
Compare
mininny
commented
Oct 29, 2024
|
|
||
| toolchain go1.22.0 | ||
|
|
||
| replace github.com/ethereum-optimism/optimism v1.4.3-0.20240125020216-6a3b1ec12399 => github.com/testinprod-io/optimism v0.0.0-20241018221952-12c4820e35fb |
There was a problem hiding this comment.
Will be reverted once the monorepo pr gets merged
zhwrd
approved these changes
Nov 6, 2024
Vui-Chee
added a commit
to okx/infra
that referenced
this pull request
Dec 13, 2024
….2.2 [tag-service-release] Tag op-signer at v1.2.2 * tag 'op-signer/v1.2.2': chore(deps): bump github.com/golang-jwt/jwt/v4 in /op-signer (ethereum-optimism#97) proxyd: add Redis cluster support (ethereum-optimism#100) op-conductor-ops: shorten status table titles (ethereum-optimism#110) Add sequencer voting column in status table (ethereum-optimism#108) chore(deps): bump github.com/btcsuite/btcd in /peer-mgmt-service feat: support un-overriding the leader with safeguards (ethereum-optimism#104) fix conductor_overrideLeader and support forced leadership transfer (ethereum-optimism#103) Make proxyd error handling more generic for different backend implementations op-conductor-ops: halt-sequencer command and pre-flight checks to force-active-sequencer (ethereum-optimism#101) chore(deps): bump github.com/rs/cors in /op-conductor-mon feat: Integrate op-node with op-signer for block payload signing (ethereum-optimism#59)
Vui-Chee
added a commit
to okx/infra
that referenced
this pull request
Dec 23, 2024
[tag-service-release] Tag proxyd at v4.13.0 * tag 'proxyd/v4.13.0': fix: don't count replacement underpriced errors (ethereum-optimism#112) chore(deps): bump github.com/golang-jwt/jwt/v4 in /op-txproxy (ethereum-optimism#96) chore(deps): bump github.com/golang-jwt/jwt/v4 in /op-signer (ethereum-optimism#97) proxyd: add Redis cluster support (ethereum-optimism#100) op-conductor-ops: shorten status table titles (ethereum-optimism#110) Add sequencer voting column in status table (ethereum-optimism#108) chore(deps): bump github.com/btcsuite/btcd in /peer-mgmt-service feat: support un-overriding the leader with safeguards (ethereum-optimism#104) fix conductor_overrideLeader and support forced leadership transfer (ethereum-optimism#103) Make proxyd error handling more generic for different backend implementations op-conductor-ops: halt-sequencer command and pre-flight checks to force-active-sequencer (ethereum-optimism#101) chore(deps): bump github.com/rs/cors in /op-conductor-mon feat: Integrate op-node with op-signer for block payload signing (ethereum-optimism#59)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Design Doc: ethereum-optimism/design-docs#103
This PR adds support for op-signer to sign block payloads from op-node for p2p gossiping.
The op-signer now handles
opsigner_signBlockPayloadRPC withBlockPayloadArgsas an argument. op-signer uses the private key from the KMS to hash the arguments and the sign it this signingHash, and returns it as the RPC response.This PR is a counterpart PR of ethereum-optimism/optimism#12325
Tests