Bugfix expired calculation (#37)

* Version 5.2.5

* [bugfix] Calculation within *clientlib.TokenInfo.expired()* now
  returns *None* as expected when no *expires_at* is available for
  a token. Previous versions would raise an exception.

* *False* instead of *None*

Author: tryptichon

CHANGELOG.md

@@ -1,3 +1,8 @@
+# v5.2.5
+
+* [bugfix] Calculation within *clientlib.TokenInfo.expired()* now returns *False* as expected when no *expires_at* 
+  is available for a token. Previous versions would raise an exception.
+
 # v5.2.4
 
 * Merge headers in AbstractAPI instead of replacing them.

src/hiro_graph_client/VERSION

@@ -1 +1 @@
-5.2.4
+5.2.5

src/hiro_graph_client/clientlib.py

@@ -555,7 +555,6 @@ def _log_communication(self, res: requests.Response, request_body: bool = True,
         :param res: The response of a request. Also contains the request.
         :param request_body: Option to disable the logging of the request_body.
         :param response_body: Option to disable the logging of the response_body.
-        :return:
         """
 
         def _log_headers(headers) -> str:
@@ -944,6 +943,7 @@ def decode_token(self) -> dict:
         """
         Return a dict with the decoded token payload. This payload contains detailed information about what this token
         has access to.
+
         :return: The dict with the decoded token payload.
         """
         base64_payload: list = self.token.split('.')
@@ -1088,6 +1088,7 @@ def __init__(self, token: str = None, refresh_token: str = None, expires_at: int
     def get_epoch_millis() -> int:
         """
         Get timestamp
+
         :return: Current epoch in milliseconds
         """
         return int(time.time_ns() / 1000000)
@@ -1129,10 +1130,14 @@ def parse_token_result(self, res: dict, what: str) -> None:
 
     def expired(self) -> bool:
         """
-        Check token expiration
+        Check token expiration.
 
-        :return: True when the token has been expired *(expires_at - refresh_offset) <= get_epoch_mills()*
+        :return: True when the token has been expired *(expires_at - refresh_offset) <= get_epoch_mills()*. If
+                 no *expires_at* is available, always return False since this token would never expire.
         """
+        if self.refresh_time() is None:
+            return False
+
         return self.refresh_time() <= self.get_epoch_millis()
 
     def fresh(self, span: int = 30000) -> bool:
@@ -1312,7 +1317,7 @@ def refresh_time(self) -> Optional[int]:
         """
         Calculate refresh time.
 
-        :return: Timestamp after which the token becomes invalid.
+        :return: Timestamp after which the token becomes invalid. Returns None if token cannot be refreshed.
         """
         return self._token_info.refresh_time()