Is there any documentation that needs to be updated along with the impl
changes?
Would it make sense to support the preference properties?
java.net.preferIPv4Stack
java.net.preferIPv6Addresses
Will the previous jdwp tests run with IPv6 or just the new additions?
Should probably have reviewers with networking expertise. core-libs(?)
Do we know if IPv6 is enabled in our test infrastructure?
A stray "printf" statement in the initial webrev. socketTransport.c
On 3/27/19, 7:04 PM, Alex Menkov wrote:
Hi all,
Please review the fix for
https://bugs.openjdk.java.net/browse/JDK-8184770
webrev:
http://cr.openjdk.java.net/~amenkov/IPv6/webrev.00/
Main changes are in socketTransport.c - the code is updated to support
both IPv4 and IPv6.
Some details to simplify reviewing:
- listening:
- if IP address is specified (like 127.0.0.1 or ::1), connector
listens only on this address;
- for backward compatibility if no address (or "localhost") is
specified, IPv4 is used (if supported by the host);
- if "*" is specified (means "listen on all local interfaces"), dual
mode socket is used to listen on both IPv6 and IPv4 addresses;
- AllowedPeerInfo structure (for "allow" option) is updated to use
IPv6 address/mask, support for IPv4 is implemented by using "mapped"
IPv4 addresses;
- attaching: agent resolves and tries to connect to all (IPv4 and
IPv6) addresses, IPv4 are tried first;
SocketListeningConnector.java/SocketTransportService.java are updated
to support IPv6 addresses (the addresses may contain colons);
new JdwpAttachTest.java/JdwpListenTest.java test that listening and
attaching works for all available addresses (Ipv4 and IPv6)
BasicJDWPConnectionTest.java was renamed to JdwpAllowTest.java (as it
tests "allow" functionality), tests for mask (prefix length)
functionality are added (for both IPv4 and IPv6);
--alex
