Skip to content

Conversation

@clemtaylor
Copy link
Contributor

37% of the default log entries for a node that has been up for ~24hrs was "version handshake timeout..."

…non-contributing peers (port scanners?)

37% of the log default log entries for a node that has been up for ~24hrs was "version handshake timeout..."
@meshcollider
Copy link
Contributor

LGTM, utACK c887f87

@fanquake fanquake added the P2P label Feb 3, 2018
@gmaxwell
Copy link
Contributor

gmaxwell commented Feb 3, 2018

utACK. Worth mentioning that the reduction in other log entries seems to make these entries more confusing to users, since now you can open your logs and see a page of nothing but this message.

@laanwj
Copy link
Member

laanwj commented Feb 6, 2018

Obvious utACK c887f87

@laanwj laanwj merged commit c887f87 into bitcoin:master Feb 6, 2018
laanwj added a commit that referenced this pull request Feb 6, 2018
…ssage

c887f87 Extend #11583 to include the most common message generated by non-contributing peers (port scanners?) 37% of the log default log entries for a node that has been up for ~24hrs was "version handshake timeout..." (Clem Taylor)

Pull request description:

  37% of the default log entries for a node that has been up for ~24hrs was "version handshake timeout..."

Tree-SHA512: dceeee5d55a9ff7570174aeb63faac9beda239087220522adefef7ed11e0eeffa008ca28726011247c8834c1a222d37817baf895635ab874a95ebc435959070e
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jan 27, 2020
…ke timeout" message

c887f87 Extend bitcoin#11583 to include the most common message generated by non-contributing peers (port scanners?) 37% of the log default log entries for a node that has been up for ~24hrs was "version handshake timeout..." (Clem Taylor)

Pull request description:

  37% of the default log entries for a node that has been up for ~24hrs was "version handshake timeout..."

Tree-SHA512: dceeee5d55a9ff7570174aeb63faac9beda239087220522adefef7ed11e0eeffa008ca28726011247c8834c1a222d37817baf895635ab874a95ebc435959070e
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants