Resolve "WS2Pv2: Implement ACK and OK messages processing"

Closes #143 (closed)

added C-ws2p label

added 4 commits

• b7c84a36...d1cca2ae - 2 commits from branch dev
• 8d2401f6 - [docs] dev: illustrate ws2p
• a4c88aa3 - [feat] ws2p: implement and test ack and ok messages

Compare with previous version

added 1 commit

• b1099a1d - cargo fmt

Compare with previous version

@HugoTrentesaux Félicitations tu a réussi plus rapidement que ce que je ne pensais à finir la négociation d'une connexion ! Je n'ai qu'une dizaine de remarques ce qui est très peu vu le fait que c'est ta 1ère contribution au code source en lui-même ^^

J'ai également des remarques générales mais bloquantes (donc qui doivent être traités) :

1. On ne fait pas de commit dédié a fmt, il faut amend ton commit :)
2. Le test actuel ne test que les cas passants. Il faudrait créer 1 ou plusieurs nouveaux tests qui vérifient que la connexion passe bien en Denial en cas te tentative d'usurpation. Il suffit par exemple de fournir au client un expected_full_id différent ;)

EDIT: Très bien ton schéma, il faudrait parler de son existence quelque par dans la doc pour développeurs :)

added 2 commits

• 9c667e40 - replace println -> debug
• 2db2cd02 - fix things

Compare with previous version

Ça m'embrouille beaucoup de faire un force push à chaque modification. Est-ce que ça te va si on change légèrement la méthode de travail ? J'aimerais faire des petits commits successifs qui prennent en compte tes relectures mais qui ne remplissent pas forcément les exigences pour être mergés. Et une fois que c'est bon, je réorganise tout, je rebase sur dev et on relit une dernière fois l'ensemble avant de merger.

À propos du test, j'ai essayé de remplacer

server_node_clone.my_key_pair.public_key(),

par

PubKey::Ed25519(keypair3().public_key()), 

mais j'avais une erreur. En cherchant un peu, j'ai fini par écrire le test trivial :

fn test_connection_negociation_denial() {
    test_connection_negociation();
}

mais il échoue avec :

test test_connection_negociation_denial ... ok
test test_connection_negociation ... FAILED
thread 'test_connection_negociation' panicked at 'Not receive server controller sender'

autrement dit, le test passe la première fois, mais pas la deuxième... étrange.

Je regarderai ça plus tard.

added 1 commit

• b287658f - moved update_status to handler

Compare with previous version