8239052: java/net/httpclient/whitebox/SSLEchoTubeTestDriver.java failed with BufferUnderflowException against TLSv1.3

[Daniel Fuchs]

Thanks Chris!

I was going to answer that this was not possible as it requires
multiple return values but I found a way:

http://cr.openjdk.java.net/~dfuchs/webrev_8239052/webrev.02/

best regards,

-- daniel

On 22/02/2020 13:53, Chris Hegarty wrote:
>> On 21 Feb 2020, at 17:41, Daniel Fuchs<daniel.fuchs at oracle.com>  wrote:
>>
>> Thanks John!
>>
>> I have verified that indeed TLSv1.3 is selected by default,
>> and updated the fix as per your suggestion:
>>
>> http://cr.openjdk.java.net/~dfuchs/webrev_8239052/webrev.01/
> Good sleuthing Daniel.
> 
> Your changes look good, just wondering if they could be factored out
> into a separate method, say getLong(ByteBuffer), so that the crux of the
> test logic in onNext would not be slightly obfuscated by this?
> 
> -Chris.