On Tue, 13 Jan 2026 18:22:49 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`
>
> Josiah Noel has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains 20 commits:
> 
>  - Merge branch 'openjdk:master' into JDK-8368695
>  - Merge branch 'openjdk:master' into JDK-8368695
>  - Merge branch 'master' into JDK-8368695
>  - Merge branch 'master' into JDK-8368695
>  - reduce diff
>  - Merge remote-tracking branch 'upstream/master' into JDK-8368695
>  - Update SwitchingProtocolTest.java
>  - Update SwitchingProtocolTest.java
>  - Update SwitchingProtocolTest.java
>  - add exception test
>  - ... and 10 more: https://git.openjdk.org/jdk/compare/f23752a7...c9684377

@vy any chance to make progress here .. this is currently somehow blocking and 
proposal seems quite lightweight...

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

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

Reply via email to