[jdk11u-dev] RFR: 8229481: sun/net/www/protocol/https/ChunkedOutputStream.java failed with a SSLException [v4]

Paul Hohensee phh at openjdk.org
Mon Jul 17 22:14:18 UTC 2023


On Mon, 17 Jul 2023 09:29:55 GMT, luchenlin <duke at openjdk.org> wrote:

>> I backport this for parity with 11.0.21-oracle.
>> test/jdk/sun/net/www/protocol/https/ChunkedOutputStream.java
>> 'testPlainText(server.getAuthority())' is not added because of TestHttpsServer does not have method 'getAuthority()'
>
> luchenlin has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fix error

There's no reason to backport testPlainText if it's not invoked. getAuthority went in as part of [JDK-8268464](https://bugs.openjdk.org/browse/JDK-8268464), for which the patch is fairly large but worth consideration. If you don't want to do that backport first, please include getAuthority from it and use it for this backport.

-------------

Changes requested by phh (Reviewer).

PR Review: https://git.openjdk.org/jdk11u-dev/pull/2037#pullrequestreview-1533776405


More information about the jdk-updates-dev mailing list