From 8ee0366203e951ea87e93e5efcd335a567ca14bf Mon Sep 17 00:00:00 2001 From: Matt Drayer Date: Tue, 14 Nov 2017 10:57:06 -0500 Subject: [PATCH] Revise OData exception logging --- common/djangoapps/third_party_auth/saml.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/common/djangoapps/third_party_auth/saml.py b/common/djangoapps/third_party_auth/saml.py index 046d71911f..f2f88c0bd6 100644 --- a/common/djangoapps/third_party_auth/saml.py +++ b/common/djangoapps/third_party_auth/saml.py @@ -247,8 +247,14 @@ class SapSuccessFactorsIdentityProvider(EdXSAMLIdentityProvider): return missing def log_bizx_api_exception(self, transaction_data, err): - sys_msg = err.response.json() if err.response else 'Not available' - headers = err.response.headers if err.response else 'Not available' + try: + sys_msg = err.response.content + except AttributeError: + sys_msg = 'Not available' + try: + headers = err.response.headers + except AttributeError: + headers = 'Not available' token_data = transaction_data.get('token_data') token_data = token_data if token_data else 'Not available' log_msg_template = (