Check the http status code before using Response

This was causing a more difficult to understand failure in the JSON
parser so let's do it the more correct way. Check the response code
before trying to parse the response content.

Signed-off-by: Eric B Munson <eric@munsonfam.org>

CHANGELOG.md

@@ -1,3 +1,7 @@
+##4.0.6
+
+Check the http status code before trying to use the Response object.
+
 ##4.0.5
 
 Fiddly bits of packaging...

src/libopensonic/_version.py

@@ -17,4 +17,4 @@
 
 
 #Semantic versioning string for the library
-__version__ = '4.0.5'
+__version__ = '4.0.6'

src/libopensonic/connection.py

@@ -2451,20 +2451,22 @@ def _doRequestWithLists(self, methodName, listMap, query=None):
         url = f"{self._baseUrl}:{self._port}/{self._serverPath}/{methodName}"
 
         if self._useGET:
-            res = requests.get(url, params=qdict)
+            res = requests.get(url, params=qdict, timeout=(60,300))
         else:
-            res = requests.post(url, data=qdict)
+            res = requests.post(url, data=qdict, timeout=(60,300))
 
         return res
 
 
     def _handleInfoRes(self, res):
         # Returns a parsed dictionary version of the result
+        res.raise_for_status()
         dres = res.json()
         return dres['subsonic-response']
 
 
     def _handleBinRes(self, res):
+        res.raise_for_status()
         contType = res.headers['Content-Type'] if 'Content-Type' in res.headers else None
 
         if contType: