-
Notifications
You must be signed in to change notification settings - Fork 304
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
Enhanced IPv6 support (libp2p) #8332
Enhanced IPv6 support (libp2p) #8332
Conversation
infrastructure/io/src/test/java/tech/pegasys/teku/infrastructure/io/IPVersionResolverTest.java
Dismissed
Show dismissed
Hide dismissed
have you tried single stack ipV6 ? Discovery won't work but we should be able to run over outbound connections only. |
Yes. Discovery still works though. It doesn't support dual-stack but works with single stack IPv6. |
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.
One thing that does not convince me is that we are leaking the feature via --p2p-advertised-ips
and --p2p-interfaces
aliases but then we hide the v6 options via --X
. Maybe you're right but i'd like to discuss it
networking/p2p/src/main/java/tech/pegasys/teku/networking/p2p/libp2p/LibP2PNetworkBuilder.java
Show resolved
Hide resolved
networking/p2p/src/main/java/tech/pegasys/teku/networking/p2p/network/config/NetworkConfig.java
Outdated
Show resolved
Hide resolved
Sorry I didn't mean to "request changes", I pressed the wrong button on intelliJ :) |
a3896ad
to
34ed694
Compare
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.
LGTM. with a doubt
...ing/p2p/src/test/java/tech/pegasys/teku/networking/p2p/network/config/NetworkConfigTest.java
Show resolved
Hide resolved
03c1ef8
to
b572dcf
Compare
PR Description
Add new options (hidden for now until discovery is adapted for dual stack as well):
--Xp2p-port-ipv6
--Xp2p-advertised-port-ipv6
Also enhanced
--p2p-interface
and--p2p-advertised-ip
to accept multiple entries to support dual-stack (IPv4 and IPv6), but for the moment didn't add changes to description until we unhide the ipv6 feature entirely.Testing (dual stack)
Response from
eth/v1/node/identity
Fixed Issue(s)
related to #8069
Documentation
doc-change-required
label to this PR if updates are required.Changelog