Skip to content
This repository has been archived by the owner on Jul 21, 2023. It is now read-only.

fix: false discovery #92

Merged
merged 2 commits into from
Mar 27, 2019
Merged

fix: false discovery #92

merged 2 commits into from
Mar 27, 2019

Commits on Mar 27, 2019

  1. fix: false discovery 

    This PR prevents the DHT claiming discovery of a peer that it may not have even discovered and also multiple discoveries of the same peer.

DHT was listening for 'peer-mux-established' from the switch and emitting a 'peer' event in response to it. Listening to 'peer-mux-established' implies that the peer has _already_ been discovered so there's no need to emit it again and moreover the peer may not have been discovered by the DHT in the first place!

License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
    Alan Shaw committed Mar 27, 2019
    Configuration menu
    Copy the full SHA
    e38a49f View commit details
    Browse the repository at this point in the history

  2. fix: tests 

    License: MIT
Signed-off-by: Alan Shaw <alan.shaw@protocol.ai>
    Alan Shaw committed Mar 27, 2019
    Configuration menu
    Copy the full SHA
    d8dc3ac View commit details
    Browse the repository at this point in the history