On Sun, 10 May 2026 23:25:55 GMT, Josiah Noel <[email protected]> wrote:

>> Remaking the PR since I messed up the upstream merge on the other one. See 
>> (https://github.com/openjdk/jdk/pull/27751) for the bulk of the previous 
>> discussion
>> 
>> - adds a flag to ExchangeImpl to signal whether the current request is a GET 
>> Upgrade request
>> - Adds a new `UpgradeInputStream`/`UpgradeOutputStream` to ensure that the 
>> server keeps track of when the upgraded request is closed
>> - on 101 response codes, `sendResponseHeaders` will not immediately close 
>> the output stream 
>> - on 101 response codes, `sendResponseHeaders` will use an 
>> `UpgradeInputStream`
>> 
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Josiah Noel has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains 34 commits:
> 
>  - Merge branch 'openjdk:master' into JDK-8368695
>  - Merge branch 'openjdk:master' into JDK-8368695
>  - Merge branch 'openjdk:master' into JDK-8368695
>  - Merge branch 'openjdk:master' into JDK-8368695
>  - lint
>  - exclude http/1.0
>  - constant
>  - will not upgrade when there is a body
>  - Update UpgradeOutputStream.java
>  - Merge branch 'openjdk:master' into JDK-8368695
>  - ... and 24 more: https://git.openjdk.org/jdk/compare/5caa357f...754e48e1

Is anything to be done here? Would be great to have this integrated into the 
next JDK release!

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

PR Comment: https://git.openjdk.org/jdk/pull/27989#issuecomment-4539980924

Reply via email to