Skip to content

Commit

Permalink
Handle the CLI flag more explicitly
Browse files Browse the repository at this point in the history
  • Loading branch information
johnsaigle committed Feb 21, 2025
1 parent 4e5570b commit e2e88f5
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions node/cmd/guardiand/node.go
Original file line number Diff line number Diff line change
Expand Up @@ -926,10 +926,12 @@ func runNode(cmd *cobra.Command, args []string) {
logger.Fatal("If coinGeckoApiKey is set, then chainGovernorEnabled must be set")
}

txVerifierChains, err := parseTxVerifierChains(*transferVerifierEnabledChains)
logger.Info("parsed txVerifierChains", zap.Any("chains", txVerifierChains))
if err != nil {
logger.Fatal("Could not parse transferVerifierEnabledChains", zap.Error(err))
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 publicRpcLogDetail common.GrpcLogDetail
Expand Down Expand Up @@ -1913,7 +1915,7 @@ func argsConsistent(args []string) bool {
return true
}

// Parse a list of chainIds from a comma separated string and validate that they have a Transfer Verifier implementation
// Parse a list of chainIds from a comma separated string and validate that they have a Transfer Verifier implementation.
func parseTxVerifierChains(
// A comma-separated list of Wormhole ChainIDs.
input string,
Expand Down

0 comments on commit e2e88f5

Please sign in to comment.