Aias00 opened a new issue, #6582:
URL: https://github.com/apache/shenyu/issues/6582

   ## Description
   The `shenyu.httpclient.responseTimeout` property is parsed and has a getter, 
but is never passed to `HttpClient.responseTimeout(...)` or any mechanism in 
the factory. The actual per-request timeout comes from 
`AbstractHttpClientPlugin.execute()` reading `Constants.HTTP_TIME_OUT` from the 
exchange attribute (set by DividePlugin from `ruleHandle.getTimeout()`).
   
   ## Location
   ```
   shenyu-plugin-httpclient/.../config/HttpClientProperties.java:62 
(responseTimeout = 3000L), 165-168 (getter)
   HttpClientFactory.java:110-143 (createInstance never calls 
getResponseTimeout())
   ```
   
   ## Impact
   Operators setting `shenyu.httpclient.responseTimeout=10000` expecting a 
global response timeout get no effect — silent misconfiguration. A route 
misconfigured with `timeout=0` yields `Duration.ofMillis(0)` = zero timeout, 
failing every request immediately.
   
   ## Suggested fix
   Apply `responseTimeout` as the HttpClient-level `responseTimeout()`, or 
deprecate/remove the dead field and document that timeouts are per-route.
   
   ## Related existing issue(s)
   None
   
   _Identified during the 2026-08-02 audit; full list in 
[`docs/issue-candidates-2026-08-02.md`](docs/issue-candidates-2026-08-02.md)._


-- 
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]

Reply via email to