fix: better handle empty sensor responses and improve logging

Author: dahlb

src/blueair_api/http_aws_blueair.py

@@ -39,17 +39,18 @@ async def request_with_errors_wrapper(*args, **kwargs) -> ClientResponse:
         status_code = response.status
         try:
             response_json = await response.json(content_type=None)
-            if "statusCode" in response_json:
-                _LOGGER.debug("response json found, checking status code from response")
-                status_code = response_json["statusCode"]
+            if response_json is not None:
+                if "statusCode" in response_json:
+                    _LOGGER.debug("response json found, checking status code from response")
+                    status_code = response_json["statusCode"]
         except Exception as e:
             _LOGGER.error(f"Error parsing response for errors {e}")
             raise e
         if status_code == 200:
             _LOGGER.debug("response 200")
             return response
         if 400 <= status_code <= 500:
-            _LOGGER.debug("auth error")
+            _LOGGER.debug(f"auth error, {status_code}")
             url = kwargs["url"]
             response_text = await response.text()
             if "accounts.login" in url:

src/blueair_api/intermediate_representation_aws.py

@@ -123,8 +123,12 @@ class SensorRecord:
 
 class SensorHistory(list[SensorRecord]):
     def __init__(self, response):
-        sensors = response[0]["sensors"]
-        datapoints = response[0]["datapoints"]
+        if response is None:
+            sensors = []
+            datapoints = []
+        else:
+            sensors = response[0]["sensors"]
+            datapoints = response[0]["datapoints"]
         sensor_records = []
         for datapoint in datapoints:
             values = {}