Integrated: 8262294: java/net/httpclient/ProxyAuthDisabledSchemes.java fails with HTTP/1.1 parser received no bytes

Daniel Jeliński djelinski at openjdk.org
Thu Mar 30 05:18:26 UTC 2023


On Wed, 29 Mar 2023 10:45:22 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:

> Please review this trivial fix for setting "Expect" header.
> 
> The original code added a new header every time. As a result, when the request was retried, it was sent with two Expect headers.
> The new code replaces the original Expect header instead of adding a new one.
> 
> Expect header allows only one value, "100-continue", so there's no risk of overwriting some other value.
> 
> No new test. I added the bug ID to the test that was intermittently failing because of this issue. Existing tests continue to pass.
> 
> Also fixed an unintentional passthrough in `switch` case that generated duplicate logs in tests.

This pull request has now been integrated.

Changeset: 2d607c9c
Author:    Daniel Jeliński <djelinski at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/2d607c9cd19bec5d4a90cb4760fba3cf83bcf982
Stats:     8 lines in 4 files changed: 1 ins; 4 del; 3 mod

8262294: java/net/httpclient/ProxyAuthDisabledSchemes.java fails with HTTP/1.1 parser received no bytes

Reviewed-by: dfuchs, jpai

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

PR: https://git.openjdk.org/jdk/pull/13224


More information about the net-dev mailing list