Skip to content

update dev#26

Merged
ca333 merged 72 commits intoGLEECBTC:devfrom
jl777:dev
Apr 24, 2018
Merged

update dev#26
ca333 merged 72 commits intoGLEECBTC:devfrom
jl777:dev

Conversation

@ca333
Copy link
Copy Markdown

@ca333 ca333 commented Apr 24, 2018

No description provided.

jl777 and others added 30 commits April 19, 2018 23:57
similar to the prior fix, if the prevblock is in the limbo state of not being in the mapIndex or not having a pindex, it used to be a severe enough treatment to blacklist the peer. However this seems too strict as with fast block production and uneven block propagation times it seems not impossible that block N+1 arrives at a node before block N does.

In such a case, if the blockheaders for N and N+1 have not already been seen, these errors were triggered. The change reduces the reaction from a DoS error blacklist points to simply rejecting the block. If it indeed is a valid block, it will soon arrive and all will be well.
remove half the redundant windows headers request fix
@ca333 ca333 merged commit 7dd47cf into GLEECBTC:dev Apr 24, 2018
ca333 pushed a commit that referenced this pull request Mar 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants