Skip to content
This repository was archived by the owner on Oct 25, 2024. It is now read-only.

Commit d8341f1

Browse files
Ruteriavalonche
authored andcommitted
Fix lint issues (#15)
1 parent 7624553 commit d8341f1

File tree

3 files changed

+13
-4
lines changed

3 files changed

+13
-4
lines changed

builder/beacon_client_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -212,10 +212,10 @@ func TestOnForkchoiceUpdate(t *testing.T) {
212212
require.NoError(t, err)
213213
require.Equal(t, PubkeyHex("0x93247f2209abcacf57b75a51dafae777f9dd38bc7053d1af526f220a7489a6d3a2753e5f3e8b1cfe39b56f43611df74b"), pubkeyHex)
214214

215-
pubkeyHex, err = bc.getProposerForNextSlot(31)
215+
_, err = bc.getProposerForNextSlot(31)
216216
require.EqualError(t, err, "slot out of sync")
217217

218-
pubkeyHex, err = bc.getProposerForNextSlot(33)
218+
_, err = bc.getProposerForNextSlot(33)
219219
require.EqualError(t, err, "slot out of sync")
220220

221221
mbn.headersCode = 404
@@ -263,6 +263,6 @@ func TestOnForkchoiceUpdate(t *testing.T) {
263263

264264
// Check proposers map error is routed out
265265
mbn.headersResp = []byte(`{ "data": [ { "header": { "message": { "slot": "65", "proposer_index": "1" } } } ] }`)
266-
slot, err = bc.onForkchoiceUpdate()
266+
_, err = bc.onForkchoiceUpdate()
267267
require.EqualError(t, err, "inconsistent proposer mapping")
268268
}

builder/service.go

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ package builder
22

33
import (
44
"errors"
5+
"fmt"
56
"net/http"
67

78
"github.com/ethereum/go-ethereum/common"
@@ -97,12 +98,20 @@ func Register(stack *node.Node, backend *eth.Ethereum, cfg *BuilderConfig) error
9798
}
9899

99100
genesisForkVersionBytes, err := hexutil.Decode(cfg.GenesisForkVersion)
101+
if err != nil {
102+
return fmt.Errorf("invalid genesisForkVersion: %w", err)
103+
}
104+
100105
var genesisForkVersion [4]byte
101106
copy(genesisForkVersion[:], genesisForkVersionBytes[:4])
102107
builderSigningDomain := boostTypes.ComputeDomain(boostTypes.DomainTypeAppBuilder, genesisForkVersion, boostTypes.Root{})
103108

104109
genesisValidatorsRoot := boostTypes.Root(common.HexToHash(cfg.GenesisValidatorsRoot))
105110
bellatrixForkVersionBytes, err := hexutil.Decode(cfg.BellatrixForkVersion)
111+
if err != nil {
112+
return fmt.Errorf("invalid bellatrixForkVersion: %w", err)
113+
}
114+
106115
var bellatrixForkVersion [4]byte
107116
copy(bellatrixForkVersion[:], bellatrixForkVersionBytes[:4])
108117
proposerSigningDomain := boostTypes.ComputeDomain(boostTypes.DomainTypeBeaconProposer, bellatrixForkVersion, genesisValidatorsRoot)

builder/validator.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,5 +45,5 @@ func (v *ValidatorPrivateData) PrepareRegistrationMessage(feeRecipientHex string
4545
if err != nil {
4646
return boostTypes.SignedValidatorRegistration{}, err
4747
}
48-
return boostTypes.SignedValidatorRegistration{msg, signature}, nil
48+
return boostTypes.SignedValidatorRegistration{Message: msg, Signature: signature}, nil
4949
}

0 commit comments

Comments
 (0)