Merge pull request #153 from SUSE-Enceladus/silence-insecure-request

Disable insecure request warning
SUSE-Enceladus · May 20, 2024 · d91381c · d91381c
2 parents 7492190 + 38544b2
commit d91381c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/lib/cloudregister/registerutils.py b/lib/cloudregister/registerutils.py
@@ -46,6 +46,10 @@
 REGISTERED_SMT_SERVER_DATA_FILE_NAME = 'currentSMTInfo.obj'
 RMT_AS_SCC_PROXY_MARKER = 'rmt_is_scc_proxy'
 
+requests.packages.urllib3.disable_warnings(
+    requests.packages.urllib3.exceptions.InsecureRequestWarning
+)
+
 
 # ----------------------------------------------------------------------------
 def add_hosts_entry(smt_server):
@@ -508,9 +512,6 @@ def get_credentials(credentials_file):
 # ----------------------------------------------------------------------------
 def refresh_registry_credentials():
     """Refresh registry credentials."""
-    # to silence InsecureRequestWarning
-    # should be fixed on a different PR
-    requests.packages.urllib3.disable_warnings()
     return get_activations()
 
 

diff --git a/lib/cloudregister/smt.py b/lib/cloudregister/smt.py
@@ -47,6 +47,11 @@ def __init__(self, smtXMLNode, https_only=False):
         if https_only:
             self._protocol = 'https'
         self._check_urls = self._form_srv_check_urls()
+        # disable InsecureRequestWarning
+        # as verification is disabled for the https request
+        requests.packages.urllib3.disable_warnings(
+            requests.packages.urllib3.exceptions.InsecureRequestWarning
+        )
 
     # --------------------------------------------------------------------
     def __eq__(self, other_smt):