From 4a176682fca4a53e6fcbe371773cd7a9e7732b17 Mon Sep 17 00:00:00 2001 From: Stephane Poignant Date: Wed, 17 Jan 2024 14:49:36 +0100 Subject: [PATCH] fix validators not registered when VC is used with web3signer (#5730) This change ensures that suggested-fee-recipient will be used as default even if the validator directory does not exist when a web3-signer-url is specified. --- beacon_chain/validator_client/common.nim | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/beacon_chain/validator_client/common.nim b/beacon_chain/validator_client/common.nim index ac7131029a..76ccbc588e 100644 --- a/beacon_chain/validator_client/common.nim +++ b/beacon_chain/validator_client/common.nim @@ -967,6 +967,10 @@ proc getFeeRecipient*(vc: ValidatorClientRef, pubkey: ValidatorPubKey, vc.config.validatorsDir, pubkey, perValidatorDefaultFeeRecipient) if staticRecipient.isOk(): Opt.some(staticRecipient.get()) + elif len(vc.config.web3SignerUrls) > 0 or len(vc.config.verifyingWeb3Signers) > 0: + # See issue 5730: getSuggestedFeeRecipient returns err if the validator directory + # does not exist, which is the case when a web3signer is used + Opt.some(perValidatorDefaultFeeRecipient) else: Opt.none(Eth1Address)