Skip to content

Commit a27a9fd

Browse files
committed
Adjust initial gen_server_call_timeout
to prevent `amqp_connection:start/2` from logging a warning when we use default values. The default for `connection_timeout` is already 60000. When we don't explicitly set it to a lower value, `amqp_connection:maybe_update_call_timeout/2` adjusts it to 70000 and logs a warning message, which may appear unexpected, especially for users upgrading to 3.8.10+ with no config changes. This change addresses that problem by pre-adjusting it to 70000, making it safe by default and ensuring our default values don't conflict with each other.
1 parent e610393 commit a27a9fd

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

deps/amqp_client/src/amqp_util.erl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ call_timeout() ->
99
undefined ->
1010
Timeout = rabbit_misc:get_env(amqp_client,
1111
gen_server_call_timeout,
12-
60000),
12+
safe_call_timeout(60000)),
1313
put(gen_server_call_timeout, Timeout),
1414
Timeout;
1515
Timeout ->

0 commit comments

Comments
 (0)