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