Fix connection infos overwrite

[Neylix]

Description

This issue fixes an problem when a node replicate an old node transaction with old connection informations (ip port, transport).
If this happen the active connection that may be good is killed and a new one is created with the old informations. So the node lose a good connection and may be stuck to download rest of summaries / transactions.

To fix this, there is a new control: if the connection status is connected we don't overwrite the current connection.

Also improve the connection when the node start, it was creating a new connection when loading all transaction in MemTableLoader init. Instead, we now wait for all synchronization to finish and then connect with the latest informations available

Type of change

• Bug fix (non-breaking change which fixes an issue)

Unit test
Start nodes, stop one and change its p2p port.
Start an other node, it should not have warning log informing a node connection is closed

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules