sync node should connect to all core nodes

skalenetwork / skale-consensus

Running the very core of SKL network, SKALE BFT consensus is universal, modern, modular, high-performance, asynchronous, provably-secure, agent-based Proof-of-Stake blockchain consensus engine in C++ 17. Includes provably secure embedded Oracle. Used by SKALE elastic blockchains. Easy and flexible enough to implement your own blockchain or smart contract platform. BLS signatures and Binary Asynchronous Consensus are main building blocks.

https://docs.skale.network/technology/consensus-spec

GNU Affero General Public License v3.0

77 stars 32 forks source link

sync node should connect to all core nodes #844

Open olehnikolaiev opened 2 weeks ago

olehnikolaiev commented 2 weeks ago

consensus decides if it should connect to any of the core nodes by verifying connections.size() + 1 < getNodeCount() this expression is wrong for sync nodes and should be changed to connections.size() + !isSyncNode() < getNodeCount()

check Schain.cpp:1126 for more details