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"

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


best regards,

-- daniel

On 13/08/2019 16:31, Daniel Fuchs wrote:

Please find below a fix for:

8229481: sun/net/www/protocol/https/ChunkedOutputStream.java
         failed with a SSLException
[1] https://bugs.openjdk.java.net/browse/JDK-8229481

[2] http://cr.openjdk.java.net/~dfuchs/webrev_8229481/webrev.01/

The test was observed failing once with a server side exception:

javax.net.ssl.SSLException: Unrecognized SSL message, plaintext connection? (see [1])

The most probable cause is that the test server received some
rogue connection from another entity.

The patch adds a hook to the server code that the test
code can override to simply ignore such rogue connections.

best regards,

-- daniel

Reply via email to