Skip to content

Commit

Permalink
Merge pull request #1037 from UniqueNetwork/fix/do-not-require-seconded
Browse files Browse the repository at this point in the history
Do not require seconded in block announce
  • Loading branch information
CertainLach authored Nov 10, 2023
2 parents 081791e + 6c79996 commit a3f68ee
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions node/cli/src/service.rs
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ use cumulus_client_consensus_aura::collators::lookahead::{
};
use cumulus_client_consensus_common::ParachainBlockImport as TParachainBlockImport;
use cumulus_client_consensus_proposer::Proposer;
use cumulus_client_network::RequireSecondedInBlockAnnounce;
use cumulus_client_service::{
build_relay_chain_interface, prepare_node_config, start_relay_chain_tasks, DARecoveryProfile,
StartRelayChainTasksParams,
Expand Down Expand Up @@ -399,8 +398,9 @@ where
.await
.map_err(|e| sc_service::Error::Application(Box::new(e) as Box<_>))?;

// Aura is sybil-resistant, collator-selection is generally too.
let block_announce_validator =
RequireSecondedInBlockAnnounce::new(relay_chain_interface.clone(), para_id);
cumulus_client_network::AssumeSybilResistance::allow_seconded_messages();

let validator = parachain_config.role.is_authority();
let prometheus_registry = parachain_config.prometheus_registry().cloned();
Expand Down

0 comments on commit a3f68ee

Please sign in to comment.