Remove buggy save token to file in __del__

This commit is contained in:
Maël Kervella 2018-05-25 14:00:06 +00:00
parent fe1ee23c7c
commit d5c23edff9

View file

@ -45,12 +45,7 @@ class Re2oAPIClient:
self.token = self._get_token_from_file() self.token = self._get_token_from_file()
except Exception: except Exception:
self.token = self._get_token_from_server() self.token = self._get_token_from_server()
def __del__(self):
try:
self._save_token_to_file() self._save_token_to_file()
except Exception:
pass
@property @property
def need_renew_token(self): def need_renew_token(self):
@ -96,9 +91,12 @@ class Re2oAPIClient:
'expiration': self.token['expiration'].isoformat() 'expiration': self.token['expiration'].isoformat()
} }
with self.token_file.open('w') as f: try:
json.dump(data, f) with self.token_file.open('w') as f:
self.token_file.chmod(stat.S_IWRITE | stat.S_IREAD) json.dump(data, f)
self.token_file.chmod(stat.S_IWRITE | stat.S_IREAD)
except Exception:
pass
def _get_token_from_server(self): def _get_token_from_server(self):
response = requests.post( response = requests.post(
@ -120,6 +118,7 @@ class Re2oAPIClient:
""" """
if self.need_renew_token: if self.need_renew_token:
self.token = self._get_token_from_server() self.token = self._get_token_from_server()
self._save_token_to_file()
return self.token['token'] return self.token['token']
def get(self, url, headers={}, params={}, *args, **kwargs): def get(self, url, headers={}, params={}, *args, **kwargs):