Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Disable misbehaving peers while seeking for best block #2537

Merged
merged 1 commit into from
Oct 10, 2016
Merged

Conversation

arkpar
Copy link
Collaborator

@arkpar arkpar commented Oct 8, 2016

Disable a peer that does not the headers we need till some progress have been made, to prevent attempts to pull data from them until download makes some progress.
This gets triggered when a peer reports high enough total difficulty, but does not provide a header for some reason.

@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. M4-core ⛓ Core client code / Rust. and removed A0-pleasereview 🤓 Pull request needs code review. A3-inprogress ⏳ Pull request is in progress. No review needed at this stage. labels Oct 8, 2016
@coveralls
Copy link

Coverage Status

Coverage increased (+0.09%) to 86.46% when pulling bd7ca10 on sync-fix into 271bcf4 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 86.418% when pulling ac79adb on sync-fix into 271bcf4 on master.

@gavofyork gavofyork added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 10, 2016
@gavofyork gavofyork merged commit 193cdb1 into master Oct 10, 2016
@gavofyork gavofyork deleted the sync-fix branch October 10, 2016 21:05
arkpar added a commit that referenced this pull request Oct 11, 2016
arkpar added a commit that referenced this pull request Oct 11, 2016
* v1.3.6

* Print backtrace on panic (#2535)

* Don't activate peers on connect; Test (#2537)

* Removing unwarps from sync module (#2551)

* Remove unwrap from client module (#2554)

* remove unwraps in client

* imporve block hash expect message

* mining perf trace

* Fixed race condition in trace import (#2555)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants