yuanbiao
/
emqx
1
0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #10792 from thalesmg/fix-webhook-bridge-health-check-clause-r50 

fix(webhook): fix empty ehttpc worker list clause on health check (r5.0)
This commit is contained in:
Zaiming (Stone) Shi 2023-05-24 08:46:59 +02:00 committed by GitHub
parent 9363ca5846 25437adf02
commit 256cb73e43
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
apps/emqx_connector/src/emqx_connector_http.erl
View File

@ -416,7 +416,9 @@ resolve_pool_worker(#{pool_name := PoolName} = State, Key) ->
on_get_status(_InstId, #{pool_name := PoolName, connect_timeout := Timeout} = State) ->
case do_get_status(PoolName, Timeout) of
ok ->
{connected, State};
connected;
{error, still_connecting} ->
connecting;
{error, Reason} ->
{disconnected, State, Reason}
end.
@ -438,7 +440,8 @@ do_get_status(PoolName, Timeout) ->
end
end,
try emqx_utils:pmap(DoPerWorker, Workers, Timeout) of
% we crash in case of non-empty lists since we don't know what to do in that case
[] ->
{error, still_connecting};
[_ | _] = Results ->
case [E || {error, _} = E <- Results] of
[] ->