-
-
Notifications
You must be signed in to change notification settings - Fork 434
fix: support dual stack on same port #8190
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
Conversation
nflaig
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems like there are build issues, otherwise lgtm
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## unstable #8190 +/- ##
============================================
- Coverage 54.25% 54.22% -0.03%
============================================
Files 843 843
Lines 63362 63401 +39
Branches 4795 4794 -1
============================================
+ Hits 34374 34377 +3
- Misses 28913 28949 +36
Partials 75 75 🚀 New features to boost your workflow:
|
Performance Report✔️ no performance regression detected Full benchmark results
|
|
🎉 This PR is included in v1.34.0 🎉 |
**Motivation** After resolving issues with the ipv6 dual stack port bug in v1.34.1, it was noted that we should make it clear our default p2p port for both IPv4 and IPv6 is now the same at port `9000`. **Description** When we introduced IPv6 support in #5758 , we used to default the port of `9090`. When supporting dual stack option on the same port via #8190 , we should make it clear in documentation this is now by default also on port `9000`.
Motivation
--port6set to--port#7116Description