refactor(IdP): log error response of SAML request

Author: Brooke-white

redshift_connector/plugin/adfs_credentials_provider.py

@@ -33,6 +33,10 @@ def form_based_authentication(self: "AdfsCredentialsProvider") -> str:
             response: "requests.Response" = requests.get(url, verify=self.do_verify_ssl_cert())
             response.raise_for_status()
         except requests.exceptions.HTTPError as e:
+            if "response" in vars():
+                _logger.debug("form_based_authentication https response: {}".format(response.text))  # type: ignore
+            else:
+                _logger.debug("form_based_authentication could not receive https response due to an error")
             _logger.error("Request for SAML assertion when refreshing credentials was unsuccessful. {}".format(str(e)))
             raise InterfaceError(e)
         except requests.exceptions.Timeout as e:

redshift_connector/plugin/azure_credentials_provider.py

@@ -74,6 +74,12 @@ def azure_oauth_based_authentication(self: "AzureCredentialsProvider") -> str:
             )
             response.raise_for_status()
         except requests.exceptions.HTTPError as e:
+            if "response" in vars():
+                _logger.debug(
+                    "azure_oauth_based_authentication https response: {}".format(response.text)  # type: ignore
+                )
+            else:
+                _logger.debug("azure_oauth_based_authentication could not receive https response due to an error")
             _logger.error("Request for authentication from Azure was unsuccessful. {}".format(str(e)))
             raise InterfaceError(e)
         except requests.exceptions.Timeout as e:

redshift_connector/plugin/browser_azure_credentials_provider.py

@@ -117,6 +117,10 @@ def fetch_saml_response(self: "BrowserAzureCredentialsProvider", token):
             response = requests.post(url, data=payload, headers=headers, verify=self.do_verify_ssl_cert())
             response.raise_for_status()
         except requests.exceptions.HTTPError as e:
+            if "response" in vars():
+                _logger.debug("fetch_saml_response https response: {}".format(response.text))  # type: ignore
+            else:
+                _logger.debug("fetch_saml_response could not receive https response due to an error")
             _logger.error("Request for authentication from Microsoft was unsuccessful. {}".format(str(e)))
             raise InterfaceError(e)
         except requests.exceptions.Timeout as e:

redshift_connector/plugin/okta_credentials_provider.py

@@ -44,6 +44,10 @@ def okta_authentication(self: "OktaCredentialsProvider") -> str:
             )
             response.raise_for_status()
         except requests.exceptions.HTTPError as e:
+            if "response" in vars():
+                _logger.debug("okta_authentication https response: {}".format(response.text))  # type: ignore
+            else:
+                _logger.debug("okta_authentication could not receive https response due to an error")
             _logger.error("Request for authentication from Okta was unsuccessful. {}".format(str(e)))
             raise InterfaceError(e)
         except requests.exceptions.Timeout as e:

redshift_connector/plugin/ping_credentials_provider.py

@@ -35,6 +35,10 @@ def get_saml_assertion(self: "PingCredentialsProvider") -> str:
             response: "requests.Response" = requests.get(url, verify=self.do_verify_ssl_cert())
             response.raise_for_status()
         except requests.exceptions.HTTPError as e:
+            if "response" in vars():
+                _logger.debug("get_saml_assertion https response: {}".format(response.text))  # type: ignore
+            else:
+                _logger.debug("get_saml_assertion could not receive https response due to an error")
             _logger.error("Request for SAML assertion when refreshing credentials was unsuccessful. {}".format(str(e)))
             raise InterfaceError(e)
         except requests.exceptions.Timeout as e: