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

js-chat does not receive always messages after starting #18

Closed
D4nte opened this issue Mar 31, 2021 · 0 comments · Fixed by #32
Closed

js-chat does not receive always messages after starting #18

D4nte opened this issue Mar 31, 2021 · 0 comments · Fixed by #32
Labels
bug Something isn't working

Comments

@D4nte
Copy link
Contributor

D4nte commented Mar 31, 2021

Problem

After a start when connecting to the fleet, sometimes js-chat does not receive any messages but is able to send messages. This is only happen when connecting to a remote node, e.g.:

npm run chat:start --scripts-prepend-node-path=auto -- --staticNode /ip4/134.209.139.210/tcp/30303/p2p/16Uiu2HAmPLe7Mzm8TsYUubgCAW1aJoeFScxrLj8ppHFivPo97bUZ --listenAddr /ip4/0.0.0.0/tcp/55123

Details

A sleep seems to have help (TODO: paste link) as it may be due to timing issues when starting the node.

A possible scenario could be:

  1. chat dials remote node
  2. Remote node subscribe to default waku topic before identify protocol is finished, chat drop subscription because it does not know remote node support waku relay protocol *
  3. identify protocol finishes
  4. chat sends subscription to remote node but remote node does not graft chat node to mesh topic
  5. Remote node does not forward messages to chat node

The reason of 4 could be due to:

  • This is a non-verified hypothesis.

Solution

  • Monitor if the issue still occurs once wakurelay, and underlying gossipsub, is never started nwaku#445 is resolved
  • If it does, investigate step 2 to understand why chat node does not add remote node to fanout map when receiving this subscribe (by having remote node in fanout map then chat should add remote node to mesh when doing step 4).
@D4nte D4nte added bug Something isn't working waku relay labels Mar 31, 2021
@D4nte D4nte added this to the waku-relay (node) milestone Mar 31, 2021
@D4nte D4nte removed this from the waku relay (node) milestone Apr 6, 2021
@bors bors bot closed this as completed in 707a019 Apr 13, 2021
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

Successfully merging a pull request may close this issue.

1 participant