gevorkyan.aa/edx-oauth2-wordpress-backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

include return value in exception logged output

Browse Source
This commit is contained in:
lpm0073 2022-10-06 13:21:39 -05:00
parent 59e04d3722
commit c7bcf95a86
1 changed files with 12 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
wp_oauth_backend/wp_oauth.py
View File

@ -211,24 +211,24 @@ class StepwiseMathWPOAuth2(BaseOAuth2):
# so that we can include logging for diagnostic purposes. # so that we can include logging for diagnostic purposes.
@property @property
def AUTHORIZATION_URL(self) -> str: def AUTHORIZATION_URL(self) -> str:
retval = f"{self.BASE_URL}/oauth/authorize" url = f"{self.BASE_URL}/oauth/authorize"
if VERBOSE_LOGGING: if VERBOSE_LOGGING:
logger.info("AUTHORIZATION_URL: {url}".format(url=retval)) logger.info("AUTHORIZATION_URL: {url}".format(url=url))
return retval return url
@property @property
def ACCESS_TOKEN_URL(self) -> str: def ACCESS_TOKEN_URL(self) -> str:
retval = f"{self.BASE_URL}/oauth/token" url = f"{self.BASE_URL}/oauth/token"
if VERBOSE_LOGGING: if VERBOSE_LOGGING:
logger.info("ACCESS_TOKEN_URL: {url}".format(url=retval)) logger.info("ACCESS_TOKEN_URL: {url}".format(url=url))
return retval return url
@property @property
def USER_QUERY(self) -> str: def USER_QUERY(self) -> str:
retval = f"{self.BASE_URL}/oauth/me" url = f"{self.BASE_URL}/oauth/me"
if VERBOSE_LOGGING: if VERBOSE_LOGGING:
logger.info("USER_QUERY: {url}".format(url=retval)) logger.info("USER_QUERY: {url}".format(url=url))
return retval return url
@property @property
def user_details(self) -> dict: def user_details(self) -> dict:
@ -256,9 +256,10 @@ class StepwiseMathWPOAuth2(BaseOAuth2):
def get_user_details(self, response) -> dict: def get_user_details(self, response) -> dict:
if not self.is_valid_get_user_details_response(response): if not self.is_valid_get_user_details_response(response):
logger.error( logger.error(
"get_user_details() - received an invalid response object of {t}. Cannot continue: {response}".format( "get_user_details() - received an invalid response object of {t}:{response} Cannot continue. Returning: {retval}".format(
t=self.get_response_type(response), t=self.get_response_type(response),
response=json.dumps(response, sort_keys=True, indent=4), response=json.dumps(response, sort_keys=True, indent=4),
retval=json.dumps(self.user_details, sort_keys=True, indent=4),
) )
) )
# if we have cached results then we might be able to recover. # if we have cached results then we might be able to recover.
@ -301,7 +302,7 @@ class StepwiseMathWPOAuth2(BaseOAuth2):
"get_user_details() - response object of {t} is not a valid wp-oauth object: {response}. Cannot continue. returning: {retval}".format( "get_user_details() - response object of {t} is not a valid wp-oauth object: {response}. Cannot continue. returning: {retval}".format(
t=self.get_response_type(response), t=self.get_response_type(response),
response=json.dumps(response, sort_keys=True, indent=4), response=json.dumps(response, sort_keys=True, indent=4),
retval=self.user_details, retval=json.dumps(self.user_details, sort_keys=True, indent=4),
) )
) )
return self.user_details return self.user_details