Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize

2021-08-19 Thread Andrey Turbanov
Hello. During investigation of results of IDEA inspections I found a redundant call to Math.min in a method jdk.internal.net.http.Http2ClientImpl#getConnectionWindowSize https://github.com/openjdk/jdk/blob/master/src/java.net.http/share/classes/jdk/internal/net/http/Http2ClientImpl.java#L240 int

Re: RFR: 8258951: java/net/httpclient/HandshakeFailureTest.java failed with "RuntimeException: Not found expected SSLHandshakeException in java.io.IOException"

2021-08-19 Thread Chris Hegarty
On Thu, 19 Aug 2021 14:02:28 GMT, Daniel Fuchs wrote: > Please find here a patch that fixes intermittent failures in > HandshakeFailureTest.java. The test expects an SSLException but finds an > AssertionError instead. The AssertionError was too strong and was fired > before the

RFR: 8258951: java/net/httpclient/HandshakeFailureTest.java failed with "RuntimeException: Not found expected SSLHandshakeException in java.io.IOException"

2021-08-19 Thread Daniel Fuchs
Please find here a patch that fixes intermittent failures in HandshakeFailureTest.java. The test expects an SSLException but finds an AssertionError instead. The AssertionError was too strong and was fired before the Http2Connection had a chance to raise the proper ALPNException. -

Integrated: 8272334: com.sun.net.httpserver.HttpExchange: Improve API doc of getRequestHeaders

2021-08-19 Thread Julia Boes
On Thu, 12 Aug 2021 13:44:23 GMT, Julia Boes wrote: > This is a doc-only fix that improves the wording of the API doc of > `getRequestHeaders()`. The other changes are trivial cleanup. This pull request has now been integrated. Changeset: 1c80f078 Author:Julia Boes URL: