-
Notifications
You must be signed in to change notification settings - Fork 286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(core-p2p): split into 3 ports for blocks / transactions / others #3896
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #3896 +/- ##
===========================================
- Coverage 97.61% 97.51% -0.10%
===========================================
Files 618 627 +9
Lines 14439 14516 +77
Branches 1726 1725 -1
===========================================
+ Hits 14095 14156 +61
- Misses 155 171 +16
Partials 189 189
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This pull request introduces 1 alert when merging 48b3fcf into 5222b5c - view on LGTM.com new alerts:
|
Summary
Split p2p into 3 different ports, so that we have specific port for blocks / transactions / others and therefore allowing to fine-tune rate limit and other networking settings outside the app (with iptables for example).
Checklist