Skip to content
This repository has been archived by the owner on Oct 28, 2021. It is now read-only.

Initiate p2p disconnect when all capabilities are disabled #5675

Merged
merged 2 commits into from
Jul 17, 2019

Conversation

halfalicious
Copy link
Contributor

Fix #5674

Send out a p2p capability disconnect packet when all capabilities are disabled. Also log the outgoing Disconnect p2p capability packet to the p2pcap logger rather than the net logger.

@codecov-io
Copy link

codecov-io commented Jul 17, 2019

Codecov Report

Merging #5675 into master will increase coverage by 0.02%.
The diff coverage is 25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5675      +/-   ##
==========================================
+ Coverage   62.99%   63.01%   +0.02%     
==========================================
  Files         350      350              
  Lines       29987    29955      -32     
  Branches     3360     3353       -7     
==========================================
- Hits        18890    18877      -13     
+ Misses       9880     9861      -19     
  Partials     1217     1217

Also log the outgoing Disconnect p2p capability packet to the p2pcap
logger rather than the net logger.
@halfalicious
Copy link
Contributor Author

Rebased to fix changelog conflict

@halfalicious halfalicious merged commit 1bbb6cf into master Jul 17, 2019
@halfalicious halfalicious deleted the session-disconnect branch July 17, 2019 15:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Aleth doesn't disconnect peer session when all capabilities are disabled
3 participants