Page MenuHomePhabricator
Differential D16075

Bypass headers anti-DoS checks for NoBan peers
ClosedPublic
Actions

Authored by PiRK on Mon, Apr 29, 16:43.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC17e17ba21d0b: Bypass headers anti-DoS checks for NoBan peers
Summary

This allows you to easily avoid downloading headers twice if you have a trusted node to connect to

This concludes backport of core#25960
https://github.com/bitcoin/bitcoin/pull/25960/commits/e5982ecdc4650e9b6de38f190f3a97d792499e2a

Test Plan

ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK created this revision.Mon, Apr 29, 16:43
Herald added a reviewer: Restricted Project. · View Herald TranscriptMon, Apr 29, 16:43
PiRK requested review of this revision.Mon, Apr 29, 16:43
teamcity edited the summary of this revision. (Show Details)Mon, Apr 29, 16:43
Harbormaster completed remote builds in B28792: Diff 47479.Mon, Apr 29, 17:05
PiRK edited the summary of this revision. (Show Details)Mon, Apr 29, 18:17
Fabien accepted this revision.Tue, Apr 30, 06:50
Fabien added a subscriber: Fabien.
Fabien added inline comments.
src/net_processing.cpp
3917 ↗(On Diff #47479)

IMO NetPermissionFlags::Download would make more sense, but OK

This revision is now accepted and ready to land.Tue, Apr 30, 06:50
Closed by commit rABC17e17ba21d0b: Bypass headers anti-DoS checks for NoBan peers (authored by Suhas Daftuar <sdaftuar@gmail.com>, committed by PiRK). · Explain WhyTue, Apr 30, 08:20
This revision was automatically updated to reflect the committed changes.
PiRK added a commit: rABC17e17ba21d0b: Bypass headers anti-DoS checks for NoBan peers.

Revision Contents
Changeset List

PathSize
src/
7 lines
test/
functional/
41 lines

Diff 47506

View Options

src/net_processing.cpp

Loading...
View Options

test/functional/p2p_headers_sync_with_minchainwork.py

Loading...