Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Add a warning if users pass --sentry or --sentry-nodes
  • Loading branch information
tomaka committed Jul 31, 2020
commit 81945d024810c5d3417e23348f243661d4c78520
14 changes: 14 additions & 0 deletions client/network/src/service.rs
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,20 @@ impl<B: BlockT + 'static, H: ExHashT> NetworkWorker<B, H> {
known_addresses.push((reserved.peer_id.clone(), reserved.multiaddr.clone()));
}

let print_deprecated_message = match &params.role {
Role::Sentry { .. } => true,
Role::Authority { sentry_nodes } if !sentry_nodes.is_empty() => true,
_ => false,
};
if print_deprecated_message {
log::warn!(
"🙇 Sentry nodes are deprecated for the time being, and the `--sentry` and
`--sentry-nodes` CLI options will be removed in a future version. The Substrate
and Polkadot networking protocol require validators to be publicly-accessible.
Please do not block access to your validator nodes."
);
}

let mut sentries_and_validators = HashSet::new();
match &params.role {
Role::Sentry { validators } => {
Expand Down