Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug: multi-hop Routing #5

Open
marv1913 opened this issue Jun 17, 2024 · 0 comments
Open

Bug: multi-hop Routing #5

marv1913 opened this issue Jun 17, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@marv1913
Copy link
Collaborator

There is an issue with the multi-hop routing functionality.
The current implementation only forwards messages if the connection to the target peer is established after the message to forward has been received by the intermediate peer.
This means that if the connection to the target peer is established before the message is received, the message will not be forwarded.

I have created the class ConnectPeersMultiHopTest, which demonstrates the bug. There are two test cases:

  1. The first test shows the bug
  2. The second test shows that the message is forwarded if the connection to the intermediate peer is established after the message to forward is received.

Expected behavior:

Peer B should forward the message to Peer C regardless of when the connection to Peer C was established (whether the connection to Peer C was established before or after the message was received by Peer B).

@marv1913 marv1913 added the bug Something isn't working label Jun 17, 2024
marv1913 added a commit that referenced this issue Jun 17, 2024
marv1913 added a commit that referenced this issue Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant