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 To be fair, the httpserver isn't super popular, so the team probably has higher priority PR's to review. I can easily see a world where I continue `/touch`ing this PR for the next couple years, nothing to do but wait patiently I suppose. ------------- PR Comment: https://git.openjdk.org/jdk/pull/27989#issuecomment-4540039513
