On Mon, 2 Mar 2026 22:00:09 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 33 commits: > > - 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 > - copyright > - ... and 23 more: https://git.openjdk.org/jdk/compare/63c1cb3a...5f4e6b3f let me know if you have any other feedback/concerns ------------- PR Comment: https://git.openjdk.org/jdk/pull/27989#issuecomment-4005882188
