From c42c99f94f53785ae424709a99cac7e347f8b621 Mon Sep 17 00:00:00 2001
From: Stefan Strigler <stefan.strigler@emqx.io>
Date: Wed, 21 Dec 2022 15:20:53 +0100
Subject: [PATCH] fix: set a timeout for tcp connect

---
 apps/emqx_bridge/src/emqx_bridge_api.erl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/apps/emqx_bridge/src/emqx_bridge_api.erl b/apps/emqx_bridge/src/emqx_bridge_api.erl
index aa11ebc5d..8696aadaa 100644
--- a/apps/emqx_bridge/src/emqx_bridge_api.erl
+++ b/apps/emqx_bridge/src/emqx_bridge_api.erl
@@ -512,11 +512,12 @@ host_and_port(webhook, #{<<"url">> := Url}) ->
 host_and_port(_Unknown, _) ->
     undefined.
 
+%% [TODO] remove in EMQX-8588 when resource manager handles things more elegantly
 test_connection(undefined) ->
     %% be friendly, it might fail later on with a 'timeout' error.
     ok;
 test_connection({Host, Port}) ->
-    case gen_tcp:connect(Host, Port, []) of
+    case gen_tcp:connect(Host, Port, [], 5000) of
         {ok, TestSocket} -> gen_tcp:close(TestSocket);
         Error -> Error
     end.