milamberspace commented on PR #6742: URL: https://github.com/apache/jmeter/pull/6742#issuecomment-5256805718
Thanks @andreaslind01 — manual testing against the built distribution (browsing-style scenario, HTTP/2, same idle-gap pattern as the original repro) is conclusive: the stale-connection failure is gone. Great fix, and a genuinely clever regression test. Two smaller, non-blocking points from going through the GUI while testing: ### 1. The HTTP Version combo doesn't reflect what each implementation actually supports In *HTTP Request* → *Advanced* (and *HTTP Request Defaults*), the *Implementation* combo offers `HttpClient4` / `HttpClient5` / `Java` / (default), and *HTTP Version* offers `HTTP/1.1` / `HTTP/2` / (empty) — independently of each other. `HttpClient4` and `HTTP/2` can both be selected together, but `HTTPHC4Impl` never reads the `httpVersion` property at all, so the request silently runs as HTTP/1.1 regardless — nothing in the UI signals that the combination is a no-op. Would be worth either: - disabling/graying out `HTTP/2` in the *HTTP Version* combo when `HttpClient4` is the selected implementation, and/or - relabeling the item to make the fallback explicit, e.g. `HTTP/2.0 (back to HTTP/1.1)` when the active implementation doesn't support it. Relevant: `httpImplementation`/`httpVersion` combos in `HttpTestSampleGui.java` and `HttpDefaultsGui.java` — currently just two independent `JComboBox`es with no listener tying one to the other's state/labels. ### 2. Advanced-tab screenshot is stale `xdocs/images/screenshots/http-request-advanced-tab.png` (referenced from `component_reference.xml`) predates this PR by several years and doesn't show the new *HTTP Version* field or the `HttpClient5` implementation choice. Worth refreshing it as part of this PR (ideally with the Metal look-and-feel, to match the rest of the JMeter docs' screenshots) so the manual covers the feature it now documents. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
