Avoid using requests's JSONDecodeError

This exception is only present in "recent" version of requests,
typically not in the version distributed by Debian bullseye. Since
requests' JSONDecodeError is in general a subclass of
json.JSONDecodeError, we use the latter, but also handle the plain
ValueError (which json.JSONDecodeError is a subclass of) because
requests might use simplejson (which uses its own JSONDecodeError, also
a subclass of ValueError).

CHANGELOG.md

@@ -6,6 +6,9 @@
 
 ### Fixed
 
+* Add compatibility with [requests](https://requests.readthedocs.io)
+  version 2.25 and higher.
+
 ### Misc
 
 * Improve test coverage by running an HTTP server to fake the Patroni API (#55

check_patroni/types.py

@@ -1,4 +1,5 @@
 from typing import Any, Callable, List, Optional, Tuple, Union
+import json
 from urllib.parse import urlparse
 
 import attr
@@ -71,7 +72,7 @@ def rest_api(self, service: str) -> Any:
 
             try:
                 return r.json()
-            except requests.exceptions.JSONDecodeError:
+            except (json.JSONDecodeError, ValueError):
                 return None
         raise nagiosplugin.CheckError("Connection failed for all provided endpoints")