Alexey Serbin has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/22984 )
Change subject: KUDU-1457 [1/n] add IPv6 support to net utils
......................................................................
KUDU-1457 [1/n] add IPv6 support to net utils
As part of adding IPv6 support, this patch is focusses on net utlities:
- Introduction of socket address structure for IPv6.
- Parse input string representing IPv4 or IPv6 address family into
HostPort.
- Parse input HostPort (IPv4, IPv6) into corresponding address struct.
- Make some basic existing tests run for IPv6 addresses.
- Add IPv6 specific trusted subnets list.
- Make JWKS and OIDC servers bind to 127.0.0.1, explicitly. This is to
ensure it binds to IPv4 address. This can be hostname based when IPv6
support is complete for webserver.
- Return error for diagnostic sockets if address family is IPv6. The
check will be removed in subsequent changes when IPv6 support for
diagnostic sockets is added.
- Corresponding unit tests are added to the following files:
* net_util-test.cc - For parsing logic, address within network, etc.
* socket-test.cc - For socket initialisation, connectivity tests, etc.
Note: This is one of multiple patches that are collectively going to
enable Kudu to support IPv6. In the meantime, it is possible to
see some IPv6 specific tests failing on the systems where IPv6 is
disabled.
Change-Id: I22c773ffb2ff44b9cd765b546d6724ec5543586f
Reviewed-on: http://gerrit.cloudera.org:8080/22984
Reviewed-by: Alexey Serbin <[email protected]>
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
---
M src/kudu/rpc/rpc-test.cc
M src/kudu/rpc/server_negotiation.cc
M src/kudu/util/mini_oidc.cc
M src/kudu/util/net/diagnostic_socket.cc
M src/kudu/util/net/dns_resolver-test.cc
M src/kudu/util/net/net_util-test.cc
M src/kudu/util/net/net_util.cc
M src/kudu/util/net/net_util.h
M src/kudu/util/net/sockaddr.cc
M src/kudu/util/net/sockaddr.h
M src/kudu/util/net/socket-test.cc
11 files changed, 807 insertions(+), 152 deletions(-)
Approvals:
Alexey Serbin: Looks good to me, approved; Verified
Abhishek Chennaka: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/22984
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I22c773ffb2ff44b9cd765b546d6724ec5543586f
Gerrit-Change-Number: 22984
Gerrit-PatchSet: 17
Gerrit-Owner: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
