fix(iotdb): remove root `request_timeout` option

Fixes https://emqx.atlassian.net/browse/EMQX-10175

Since IoTDB bridge is not used as an authn/authz source and reuses the
webhook connector, we should have only one request timeout field (the
`resource_opts.request_ttl` one) like the webhook bridge does, to
avoid confusion and potentially bad configuration combinations.
This commit is contained in:
Thales Macedo Garitezi 2023-06-09 09:29:48 -03:00
parent 20a7ce64cd
commit 40675843a1
3 changed files with 1 additions and 15 deletions

View File

@ -141,14 +141,6 @@ request_config() ->
default => 2, default => 2,
desc => ?DESC("config_max_retries") desc => ?DESC("config_max_retries")
} }
)},
{request_timeout,
mk(
emqx_schema:timeout_duration_ms(),
#{
default => <<"15s">>,
desc => ?DESC("config_request_timeout")
}
)} )}
]. ].

View File

@ -132,7 +132,7 @@ bridge_config(TestCase, _TestGroup, Config) ->
" username = \"root\"\n" " username = \"root\"\n"
" password = \"root\"\n" " password = \"root\"\n"
" }\n" " }\n"
"iotdb_version = \"~s\"\n" " iotdb_version = \"~s\"\n"
" pool_size = 1\n" " pool_size = 1\n"
" resource_opts = {\n" " resource_opts = {\n"
" health_check_interval = 5000\n" " health_check_interval = 5000\n"

View File

@ -59,12 +59,6 @@ config_max_retries.desc:
config_max_retries.label: config_max_retries.label:
"""HTTP Request Max Retries""" """HTTP Request Max Retries"""
config_request_timeout.desc:
"""HTTP request timeout."""
config_request_timeout.label:
"""HTTP Request Timeout"""
desc_config.desc: desc_config.desc:
"""Configuration for Apache IoTDB bridge.""" """Configuration for Apache IoTDB bridge."""