Ref #772 - Fix bug in sectxt canonical check

internetstandards · Aug 3, 2023 · 8df8854 · 8df8854
1 parent 6b9c852
commit 8df8854
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/checks/http_client.py b/checks/http_client.py
@@ -92,11 +92,13 @@ def http_get_ip(
     session = requests.session()
     session.mount(f"https://{hostname}", ForcedIPHTTPSAdapter(dest_ip=ip))
     if https:
-        url = f"https://{hostname}:{port}/{path}"
+        port_suffix = "" if port == 443 else f":{port}"
+        url = f"https://{hostname}{port_suffix}/{path}"
     else:
         if ":" in ip:
             ip = f"[{ip}]"
-        url = f"http://{ip}:{port}/{path}"
+        port_suffix = "" if port == 80 else f":{port}"
+        url = f"http://{ip}{port_suffix}/{path}"
     headers["Host"] = hostname
     return http_get(url, verify=False, headers=headers, session=session, *args, **kwargs)