[ZOOKEEPER-1460] IPv6 literal address not supported for quorum members

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.4.3
• Fix Version/s: 3.5.2, 3.6.0
• Component/s: quorum
• Hadoop Flags: Reviewed
• Release Note: IPv6 addresses are now properly parsed in the config
• Assignee: Joseph Walton
• Reporter: Chris Dolan
• Created: 30/Apr/12 19:49
• Updated: 21/Jul/16 20:18
• Resolved: 23/Jun/16 20:21

Description

Via code inspection, I see that the "server.nnn" configuration key does not support literal IPv6 addresses because the property value is split on ":". In v3.4.3, the problem is in QuorumPeerConfig:

String parts[] = value.split(":");
InetSocketAddress addr = new InetSocketAddress(parts[0],
                        Integer.parseInt(parts[1]));

In the current trunk (http://svn.apache.org/viewvc/zookeeper/trunk/src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java?view=markup) this code has been refactored into QuorumPeer.QuorumServer, but the bug remains:

String serverClientParts[] = addressStr.split(";");
String serverParts[] = serverClientParts[0].split(":");
addr = new InetSocketAddress(serverParts[0],
                        Integer.parseInt(serverParts[1]));

This bug probably affects very few users because most will naturally use a hostname rather than a literal IP address. But given that IPv6 addresses are supported for clients via ZOOKEEPER-667 it seems that server support should be fixed too.

Issue Links

• relates to: ZOOKEEPER-2452 Back-port ZOOKEEPER-1460 to 3.4 for IPv6 literal address support.