Don't fail, but issue a warning when detecting redirect fails

vlaci · Feb 28, 2023 · 2840165 · 2840165
1 parent 913e382
commit 2840165
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/openconnect_sso/authenticator.py b/openconnect_sso/authenticator.py
@@ -58,9 +58,13 @@ async def authenticate(self, display_mode):
     def _detect_authentication_target_url(self):
         # Follow possible redirects in a GET request
         # Authentication will occur using a POST request on the final URL
-        response = self.session.get(self.host.vpn_url)
-        response.raise_for_status()
-        self.host.address = response.url
+        try:
+            response = self.session.get(self.host.vpn_url)
+            response.raise_for_status()
+            self.host.address = response.url
+        except Exception:
+            logger.warn("Failed to check for redirect")
+            self.host.address = self.host.vpn_url
         logger.debug("Auth target url", url=self.host.vpn_url)
 
     def _start_authentication(self):