[bisq-network/bitcoinj] Add check to not duplicate peers (#14)

Manfred Karrer notifications at github.com
Mon Sep 24 01:12:07 UTC 2018


The inactives collection contained duplicated peerAddresses after
connection loss and reconnect.
We add a check to see if the to-get-added peerAddress is not already
in the collection and only add it if it is absent.
This problem was not discovered when using the public network as the
chance that same peerAddress get reported is pretty low. But with our
provided nodes we got frequently duplicates.
    
Fixes https://github.com/bisq-network/bisq/issues/1703
You can view, comment on, or merge this pull request online at:

  https://github.com/bisq-network/bitcoinj/pull/14

-- Commit Summary --

  * Add check to not duplicate peers
  * Set to version 0.14.4.9

-- File Changes --

    M core/pom.xml (2)
    M core/src/main/java/org/bitcoinj/core/PeerGroup.java (20)
    M examples/pom.xml (2)
    M pom.xml (2)
    M tools/pom.xml (2)
    M wallettemplate/pom.xml (2)

-- Patch Links --

https://github.com/bisq-network/bitcoinj/pull/14.patch
https://github.com/bisq-network/bitcoinj/pull/14.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/bisq-network/bitcoinj/pull/14
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20180923/91bf5daa/attachment-0001.html>


More information about the bisq-github mailing list