Skip to content

Commit 17bc65c

Browse files
committed
Adjust TrafficServer timeouts to work for longer response times.
We had only been testing shorter timeouts in our test suite, but it turns out to handle longer timeouts, we need to increase the other TrafficServer timeouts to basically match the nginx read timeout (otherwise TrafficServer was still timing out after the connect timeout and retrying the request multiple times).
1 parent ace38b9 commit 17bc65c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/api-umbrella/cli/read_config.lua

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -332,9 +332,9 @@ local function set_computed_config()
332332
-- router is what should trigger the initial timeout. This also prevents the
333333
-- proxies further back in the stack from thinking the client unexpectedly
334334
-- hung up on the request.
335-
config["trafficserver"]["_connect_attempts_timeout"] = config["nginx"]["proxy_connect_timeout"] + 1
335+
config["trafficserver"]["_connect_attempts_timeout"] = config["nginx"]["proxy_read_timeout"] + 1
336336
config["trafficserver"]["_transaction_no_activity_timeout_out"] = config["nginx"]["proxy_read_timeout"] + 1
337-
config["trafficserver"]["_transaction_no_activity_timeout_in"] = config["nginx"]["proxy_send_timeout"] + 1
337+
config["trafficserver"]["_transaction_no_activity_timeout_in"] = config["nginx"]["proxy_read_timeout"] + 1
338338
config["nginx"]["_initial_proxy_connect_timeout"] = config["nginx"]["proxy_connect_timeout"] + 2
339339
config["nginx"]["_initial_proxy_read_timeout"] = config["nginx"]["proxy_read_timeout"] + 2
340340
config["nginx"]["_initial_proxy_send_timeout"] = config["nginx"]["proxy_send_timeout"] + 2

0 commit comments

Comments
 (0)