drop bad peers #341

Closed
wowario wants to merge 46 commits from <deleted>:buttholes into master
Owner
There is no content yet.
wowario added 6 commits 4 years ago
400b71b716
protocol: detect and drop asshole peers
5143a851bb
p2p: use /16 filtering on IPv4-within-IPv6 addresses
continuous-integration/drone/pr Build is passing Details
1bc5579902
p2p: remove banned peers from the white list
wowario added 1 commit 4 years ago
continuous-integration/drone/pr Build is passing Details
38b9be84f2
more aggressive testing
selsta reviewed 4 years ago
{
MINFO(context << "Peer " << context.m_connection_id << " relayed the block we were waiting on");
context.m_score += 1;
if (context.m_score > 10) // prevent a node from being all nice for a while then switching to asshole
selsta commented 4 years ago

Setting this to 10 makes it less aggressive. I would stay on 5.

Setting this to 10 makes it less aggressive. I would stay on 5.
wowario closed this pull request 4 years ago
selsta reviewed 4 years ago
{
if (!m_p2p->for_connection(uuid, [&](cryptonote_connection_context& ctx, nodetool::peerid_type peer_id, uint32_t f)->bool{
MINFO(ctx << "dropping bad peer (score " << ctx.m_score << ")");
drop_connection(ctx, 10, false);
selsta commented 4 years ago
Looks like wrong version of this patch, it should look like this: https://github.com/monero-project/monero/pull/6936/files#diff-d32229ecf55d414dcca43b6918ad711e1ffd313ef851492ecba1865323007f0fR1733
All checks were successful
continuous-integration/drone/pr Build is passing
This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: wownero/wownero#341
Loading…
There is no content yet.