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

Sort peers & fix bug with filtered peer status #354

Merged
merged 5 commits into from
Oct 30, 2017
Merged

Sort peers & fix bug with filtered peer status #354

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d7da568
Sort peers by Height DESC, Delay ASC
alexbarnsley Oct 17, 2017
e1f03cd
Removed duplicated filter of status
alexbarnsley Oct 17, 2017
2d40d87
Pass filtered peers list to findGoodPeer too
alexbarnsley Oct 18, 2017
f10d0c3
Get current peer version to filter peers based on that
alexbarnsley Oct 18, 2017
f168b79
Check version response is successful
alexbarnsley Oct 19, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 16 additions & 4 deletions client/app/src/services/network.service.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -268,10 +268,17 @@
if (!network.forcepeer) {
getFromPeer("/api/peers").then(function(response) {
if (response.success) {
storageService.set("peers", response.peers.filter(function(peer) {
return peer.status == "OK";
}));
findGoodPeer(response.peers, 0);
getFromPeer('/api/peers/version').then(function(versionResponse) {
if (versionResponse.success) {
let peers = response.peers.filter(function(peer) {
return peer.status == "OK" && peer.version === versionResponse.version;
});
storageService.set("peers", peers);
findGoodPeer(peers, 0);
} else {
findGoodPeer(storageService.get("peers"), 0);
}
});
} else {
findGoodPeer(storageService.get("peers"), 0);
}
Expand All @@ -286,6 +293,11 @@
//peer.ip=network.peerseed;
return;
}
if (index === 0) {
peers = peers.sort(function(a, b) {
return b.height - a.height || a.delay - b.delay;
});
}
peer.ip = "http://" + peers[index].ip + ":" + peers[index].port;
getFromPeer("/api/blocks/getheight").then(function(response) {
if (response.success && response.height < peer.height) {
Expand Down