Skip to content

Commit

Permalink
Ensure that txVerifier CLI flag works as expected
Browse files Browse the repository at this point in the history
  • Loading branch information
johnsaigle committed Feb 21, 2025
1 parent e2e88f5 commit 4a9bba9
Showing 1 changed file with 9 additions and 4 deletions.
13 changes: 9 additions & 4 deletions node/cmd/guardiand/node.go
Original file line number Diff line number Diff line change
Expand Up @@ -281,7 +281,10 @@ var (

subscribeToVAAs *bool

// CLI parameter value
transferVerifierEnabledChains *string
// CLI parameter value
txVerifierChains []vaa.ChainID
)

func init() {
Expand Down Expand Up @@ -927,10 +930,12 @@ func runNode(cmd *cobra.Command, args []string) {
}

if cmd.Flags().Changed("transferVerifierEnabledChains") {
txVerifierChains, err := parseTxVerifierChains(*transferVerifierEnabledChains)
logger.Info("parsed txVerifierChains", zap.Any("chains", txVerifierChains))
if err != nil {
logger.Fatal("Could not parse transferVerifierEnabledChains", zap.Error(err))
var parseErr error
// NOTE: avoid shadowing txVerifierChains here. It should refer to the global variable.
txVerifierChains, parseErr = parseTxVerifierChains(*transferVerifierEnabledChains)
logger.Debug("parsed txVerifierChains", zap.Any("chains", txVerifierChains))
if parseErr != nil {
logger.Fatal("Could not parse transferVerifierEnabledChains", zap.Error(parseErr))
}
}

Expand Down

0 comments on commit 4a9bba9

Please sign in to comment.