Skip to content

Commit

Permalink
cmd/geth: print warning when whisper config is present in toml (ether…
Browse files Browse the repository at this point in the history
…eum#21544)

* cmd/geth: print warning when whisper config is present in toml

* Update cmd/geth/config.go

Co-authored-by: Martin Holst Swende <[email protected]>
  • Loading branch information
gballet and holiman authored Sep 10, 2020
1 parent 1167639 commit d7f02b4
Showing 1 changed file with 15 additions and 0 deletions.
15 changes: 15 additions & 0 deletions cmd/geth/config.go
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,19 @@ type ethstatsConfig struct {
URL string `toml:",omitempty"`
}

// whisper has been deprecated, but clients out there might still have [Shh]
// in their config, which will crash. Cut them some slack by keeping the
// config, and displaying a message that those config switches are ineffectual.
// To be removed circa Q1 2021 -- @gballet.
type whisperDeprecatedConfig struct {
MaxMessageSize uint32 `toml:",omitempty"`
MinimumAcceptedPOW float64 `toml:",omitempty"`
RestrictConnectionBetweenLightClients bool `toml:",omitempty"`
}

type gethConfig struct {
Eth eth.Config
Shh whisperDeprecatedConfig
Node node.Config
Ethstats ethstatsConfig
}
Expand Down Expand Up @@ -117,6 +128,10 @@ func makeConfigNode(ctx *cli.Context) (*node.Node, gethConfig) {
if err := loadConfig(file, &cfg); err != nil {
utils.Fatalf("%v", err)
}

if cfg.Shh != (whisperDeprecatedConfig{}) {
log.Warn("Deprecated whisper config detected. Whisper has been moved to github.com/ethereum/whisper")
}
}

// Apply flags.
Expand Down

0 comments on commit d7f02b4

Please sign in to comment.