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

p2p: do not add recently failed addresses to the peerlist #6295

Merged
merged 1 commit into from Mar 27, 2020

Conversation

@moneromooo-monero
Copy link
Collaborator

moneromooo-monero commented Jan 15, 2020

No description provided.

@@ -1909,7 +1919,7 @@ namespace nodetool

LOG_DEBUG_CC(context, "REMOTE PEERLIST: remote peerlist size=" << peerlist_.size());
LOG_DEBUG_CC(context, "REMOTE PEERLIST: " << ENDL << print_peerlist_to_string(peerlist_));
return m_network_zones.at(context.m_remote_address.get_zone()).m_peerlist.merge_peerlist(peerlist_);
return m_network_zones.at(context.m_remote_address.get_zone()).m_peerlist.merge_peerlist(peerlist_ [this](const peerlist_entry &pe) { return !is_addr_recently_failed(pe.adr); });

This comment has been minimized.

Copy link
@stoffu

stoffu Jan 16, 2020

Contributor

Syntax error due to missing comma after peerlist_

@moneromooo-monero moneromooo-monero force-pushed the moneromooo-monero:cf branch from 9b96893 to 640dfff Jan 16, 2020
@vtnerd
vtnerd approved these changes Mar 8, 2020
@moneromooo-monero moneromooo-monero force-pushed the moneromooo-monero:cf branch from 640dfff to bcae95a Mar 20, 2020
@selsta
selsta approved these changes Mar 20, 2020
@Snipa22 Snipa22 merged commit 3ed5e7c into monero-project:master Mar 27, 2020
6 checks passed
6 checks passed
build-macos
Details
build-windows
Details
build-ubuntu
Details
libwallet-ubuntu
Details
test-ubuntu
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.