FSchumacher commented on pull request #673:
URL: https://github.com/apache/jmeter/pull/673#issuecomment-939264871
I think, you misread the docs. The *porting* does not mean the *port* part,
but the recompilation/adaption of the java source, when using the Java API with
IPv6.
What I meant is, that currently we can give JMeter an IP address plus a port
in one string ("1.2.3.4:80" for IPv4 1.2.3.4 with port 80), which as you found
out is problematic, when you specify an IPv6 address (as it contains colons,
which are used currently to separate ip and port). Your solution drops the port
part completely, which is bad (in my opinion) and probably unneeded.
What I have seen is, that if you want to specify an IPv6 plus port
combination, you put the IPv6 address in square brackets like this:
"[0::1]:80". If we use this standard notation, the parsing will be simplified
to looking at the first non space char. If it is a opening square bracket
('['), we know, that the user wants to specify an IPv6 address. We can look for
the closing bracket extract the IPv6 and are happy. (As a side effect, we can
drop adding the new library and ask the normal Java API to confirm the validity
of the IPv6 address)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
