On Wed, 5 Aug 2026 12:24:01 GMT, Jaikiran Pai <[email protected]> wrote:
>> This adds a ServerSocket-based regression test for the case where an >> unknown-length body publisher emits a single empty buffer. >> The test verifies that the server observes exactly one HTTP/1.1 terminal >> chunk. >> No production code or HTTP/2/HTTP/3 behavior is changed. >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > On a general note, I don't have a strong preference, but given the kind of > checks and errors being thrown from this test, it might be simpler to just > create a JUnit test instead of a main() based test. The existing tests in the > `test/jdk/java/net/httpclient` directory have several examples of how to > create a test that uses `junit` jtreg test action. Thank you @jaikiran for helping me find my way through my first OpenJDK contribution. And thank you @dfuch for making me think more carefully about how to write better code. Thank you both! ------------- PR Comment: https://git.openjdk.org/jdk/pull/32209#issuecomment-5394539030
