Integrated: 8304963: HttpServer closes connection after processing HEAD after JDK-7026262
    Daniel Jeliński 
    djelinski at openjdk.org
       
    Tue Mar 28 06:15:43 UTC 2023
    
    
  
On Mon, 27 Mar 2023 18:37:58 GMT, Daniel Jeliński <djelinski at openjdk.org> wrote:
> This PR fixes a regression introduced in [JDK-7026262](https://bugs.openjdk.org/browse/JDK-7026262); we started closing the exchange instead of the output stream, and exchange closes the connection when wrapped stream is not set. As an unintended side effect, the server started closing connections after handling a HEAD request.
> 
> The proposed change is to set the wrapped stream to a zero-length FixedLengthOutputStream before closing the exchange. This allows the connection to be reused.
> 
> The attached test verifies if 2 consecutive HEAD requests use the same connection. It succeeds with the proposed change, fails without it. All other tier1-3 tests continue to pass.
This pull request has now been integrated.
Changeset: 3c4cd50e
Author:    Daniel Jeliński <djelinski at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/3c4cd50e3cef5905d6c5dacddd6759e118bc50ca
Stats:     12 lines in 2 files changed: 3 ins; 5 del; 4 mod
8304963: HttpServer closes connection after processing HEAD after JDK-7026262
Reviewed-by: dfuchs
-------------
PR: https://git.openjdk.org/jdk/pull/13193
    
    
More information about the net-dev
mailing list