From d5d085289037e3eb3d970b21b32cb6df26206c88 Mon Sep 17 00:00:00 2001 From: Jarrod Johnson Date: Fri, 7 Oct 2022 09:22:37 -0400 Subject: [PATCH] Tighten redfish check timeout The default timeout is overkill in the nodediscover scenario. Notably, we can receive replies from unreachable IP addresses, and those will extend rescan to the full timeout. The devices should comfortably reply within 3 seconds, making scans exit in a timely fashion. --- confluent_server/confluent/discovery/protocols/ssdp.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/confluent_server/confluent/discovery/protocols/ssdp.py b/confluent_server/confluent/discovery/protocols/ssdp.py index 6600ca2d..1db70b10 100644 --- a/confluent_server/confluent/discovery/protocols/ssdp.py +++ b/confluent_server/confluent/discovery/protocols/ssdp.py @@ -401,7 +401,7 @@ def check_fish(urldata, port=443, verifycallback=None): verifycallback = lambda x: True url, data = urldata try: - wc = webclient.SecureHTTPConnection(_get_svrip(data), port, verifycallback=verifycallback) + wc = webclient.SecureHTTPConnection(_get_svrip(data), port, verifycallback=verifycallback, timeout=3) peerinfo = wc.grab_json_response(url) except socket.error: return None