From ef32eb01402f8323aba6ab9234bfae34545b4951 Mon Sep 17 00:00:00 2001 From: Athanasius <github@miggy.org> Date: Thu, 26 Aug 2021 15:16:42 +0100 Subject: [PATCH] CAPI: Remove un-needed check/setting of Session.retrying --- companion.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/companion.py b/companion.py index 523ff89d..4950b74e 100644 --- a/companion.py +++ b/companion.py @@ -770,12 +770,6 @@ class Session(object): # TODO: Translation ? raise ServerError("Frontier CAPI down for maintenance") from e - # TODO: Let caller decide on this - # if self.retrying: # Refresh just succeeded but this query failed! Force full re-authentication - # self.retrying = False - # raise CredentialsError('query failed after refresh') from e - - # TODO: Better to return error and have upstream re-try auth ? logger.exception('Frontier CAPI: Misc. Error') raise ServerError('Frontier CAPI: Misc. Error') from e @@ -788,8 +782,6 @@ class Session(object): # LANG: Frontier CAPI data retrieval failed raise ServerError(f'{_("Frontier CAPI query failure")}: {capi_endpoint}') from e - self.retrying = False - if capi_endpoint == self.FRONTIER_CAPI_PATH_PROFILE and 'commander' not in capi_data: logger.error('No commander in returned data')