fix: guard get_error_reason against non-JSON responses

Author: fguillot

miniflux.py

@@ -48,10 +48,11 @@ def get_error_reason(self) -> str:
         Returns:
             str: The error message from the response body, or a default message if not available.
         """
-        result = self._response.json()
         default_reason = f"status_code={self.status_code}"
-        if isinstance(result, dict):
-            return result.get("error_message", default_reason)
+        if self._response.headers.get("Content-Type") == "application/json":
+            result = self._response.json()
+            if isinstance(result, dict):
+                return result.get("error_message", default_reason)
         return default_reason
 
 
@@ -133,14 +134,10 @@ def __init__(
             ValueError: If neither `api_key` nor both `username` and `password` are provided.
         """
         if not base_url.startswith(("http://", "https://")):
-            raise ValueError(
-                "base_url must be a valid URL starting with http:// or https://"
-            )
+            raise ValueError("base_url must be a valid URL starting with http:// or https://")
 
         if not api_key and not (username and password):
-            raise ValueError(
-                "Either api_key or both username and password must be provided"
-            )
+            raise ValueError("Either api_key or both username and password must be provided")
 
         self._base_url = base_url.rstrip("/")
         self._timeout = timeout
@@ -391,9 +388,7 @@ def get_icon_by_feed_id(self, feed_id: int) -> dict:
         """
         return self.get_feed_icon(feed_id)
 
-    def create_feed(
-        self, feed_url: str, category_id: Optional[int] = None, **kwargs
-    ) -> int:
+    def create_feed(self, feed_url: str, category_id: Optional[int] = None, **kwargs) -> int:
         """
         Create a new feed.
 
@@ -659,9 +654,7 @@ def get_entries(self, **kwargs) -> dict:
             return response.json()
         self._handle_error_response(response)
 
-    def update_entry(
-        self, entry_id: int, title: Optional[str] = None, content: Optional[str] = None
-    ) -> dict:
+    def update_entry(self, entry_id: int, title: Optional[str] = None, content: Optional[str] = None) -> dict:
         """
         Update an entry.
 
@@ -781,9 +774,7 @@ def get_enclosure(self, enclosure_id: int) -> dict:
             return response.json()
         self._handle_error_response(response)
 
-    def update_enclosure(
-        self, enclosure_id: int, media_progression: Optional[int] = None
-    ) -> bool:
+    def update_enclosure(self, enclosure_id: int, media_progression: Optional[int] = None) -> bool:
         """
         Update an enclosure.