Fix: I broke it

Author: Epse

src/obelisk/asynchronous/client.py

@@ -76,6 +76,7 @@ async def _get_token(self):
 
         async with httpx.AsyncClient() as client:
             response = None
+            request = None
             last_error = None
             retry = self.retry_strategy.make()
             while not response:
@@ -87,7 +88,6 @@ async def _get_token(self):
                         headers=headers)
 
                     response = request.json()
-                    break
                 except Exception as e:
                     last_error = e
                     self.log.error(e)
@@ -96,10 +96,8 @@ async def _get_token(self):
                     else:
                         break
 
-            if response is None and last_error is not None:
-                raise last_error
-
-            raise ObeliskError("No response received")
+            if not response or not request:
+                raise (last_error if last_error is not None else ObeliskError("No response"))
 
             if request.status_code != 200:
                 if 'error' in response:
@@ -148,7 +146,7 @@ async def http_post(self, url: str, data: Any = None,
             response = None
             retry = self.retry_strategy.make()
             last_error = None
-            while not response or await retry.should_retry():
+            while not response:
                 if response is not None:
                     self.log.debug(f"Retrying, last response: {response.status_code}")
 
@@ -164,7 +162,10 @@ async def http_post(self, url: str, data: Any = None,
                 except Exception as e:
                     self.log.error(e)
                     last_error = e
-                    continue
+                    if await retry.should_retry():
+                        continue
+                    else:
+                        break
 
             if not response and last_error:
                 raise last_error