[teststabilization] RFR: 8229481: sun/net/www/protocol/https/ChunkedOutputStream.java failed with a SSLException

Daniel Fuchs daniel.fuchs at oracle.com
Thu Aug 15 15:26:43 UTC 2019


Hi Michael,

On 15/08/2019 13:57, Michael McMahon wrote:
> Would it be clearer if test0() of chunkedoutputstream was renamed to 
> testPlainText() and it is given the authority component only. So, it 
> would create the URL itself and it would be clearer that it is 
> creating a http url?
> 
> Minor comment typo in "veryfy"

Done and done.

http://cr.openjdk.java.net/~dfuchs/webrev_8229481/webrev.03

Thanks for the feedback!

-- daniel

> 
> Otherwise, looks good.
> 
> - Michael.
> 
> On 14/08/2019 16:00, Daniel Fuchs wrote:
>> Michael reminded me that the first byte read off a TLS connection
>> is supposed to be 0x16.
>>
>> So here is a cleaner alternative: a
>>
>> http://cr.openjdk.java.net/~dfuchs/webrev_8229481/webrev.02
>>
>> best regards,
>>
>> -- daniel



More information about the net-dev mailing list