Looks fine to me.
hostname.indexOf('.') == -1
May be more straightforward to use hostname.contains(".").
Thanks,
Xuelei
On 5/14/2016 3:56 AM, Rajan Halade wrote:
> Please provide your review of this one line fix.
>
> These tests fail on machine where non-FQDN hostname is returned. As per
> RFC for SNI extension, hostname needs to be fully qualified DNS hostname
> and same is asserted in JDK when it adds hostname to SNI extension. This
> fix skips test execution if hostname returned is not fully qualified.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8155049
> Webrev: http://cr.openjdk.java.net/~rhalade/8155049/webrev.00/
>
> Thanks,
> Rajan
