[bisq-network/bitcoinj] Connection handling fixes (#31)
Oscar Guindzberg
notifications at github.com
Tue Apr 2 20:07:09 UTC 2019
See https://github.com/bisq-network/bitcoinj/issues/30 for details
You can view, comment on, or merge this pull request online at:
https://github.com/bisq-network/bitcoinj/pull/31
-- Commit Summary --
* PeerGroup.handlePeerDeath(): check isAlreadyAdded() before calling inactives.offer(address).
* PeerGroup.triggerConnectionsJob: Remove isAlreadyAdded() check because it is not necessary.
* PeerGroup.addInactive(): Remove isAlreadyAdded() check because it is not necessary.
* PeerAddress: add equalsIgnoringMetadata() method
* PeerGroup.addInactive(): Ignore metadata fields when checking for duplicates
* PeerGroup.isAlreadyAdded(): refactor to ignore metadata and inline method because it is only used once.
* PeerGroup.triggerConnectionsJob code clean up.
* PeerGroup.handlePeerDeath() don't check isRunning(). PeerGroup.stopAsync() set downloadPeer to null before channels are stopped instead.
* BlockingClient.writeBytes() error handling improvement.
-- File Changes --
M core/src/main/java/org/bitcoinj/core/PeerAddress.java (16)
M core/src/main/java/org/bitcoinj/core/PeerGroup.java (47)
M core/src/main/java/org/bitcoinj/net/BlockingClient.java (5)
-- Patch Links --
https://github.com/bisq-network/bitcoinj/pull/31.patch
https://github.com/bisq-network/bitcoinj/pull/31.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/31
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.bisq.network/pipermail/bisq-github/attachments/20190402/905b30ff/attachment.html>
More information about the bisq-github
mailing list