Greetings all, I've fixed this and created a test to cover it, is there a sponsor who could push this through? Here's a link to the webrev https://googledrive.com/host/0B2CI6Ih--1t5bVVwbVlBRmpVMDg/index.html.
It's a simple fix that correctly consumes the bytes from a SOCKS reply which represent an IPv6 address or a domain name. I also had to fix SocksServer as it was not correctly constructing a String representation of an IPv6 address from byte[]. I didn't find it necessary to cover the case of DOMAIN_NAME in the test as name resolution happens locally and not on the SOCKS server, which is perhaps worth another fix. Best, Dimitar Mavrodiev
