fix: add AsyncResponse type to niquests response object (#439)

Changes proposed in this pull request:

* the object returned by niquests in the response can be `AsyncResponse`
(child class of `Response`:
https://github.com/jawah/niquests/blob/6dc75e944c41551a90ccabf08a203f1a719ce87b/src/niquests/models.py#L1660)
if the request calls were made async-ly:
https://github.com/jawah/niquests/blob/6dc75e944c41551a90ccabf08a203f1a719ce87b/src/niquests/async_session.py#L1150-L1182

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Refactor**
* Updated error handling to work consistently with both synchronous and
asynchronous response objects, including broader support in the core
exception classes and `check_error`.
* **Chores**
* Tightened the minimum supported `niquests` version requirement to
`3.4.2` (still under the same major-version cap).
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Signed-off-by: kyteinsky <kyteinsky@gmail.com>
Signed-off-by: Oleksandr Piskun <oleksandr2088@icloud.com>
Co-authored-by: Oleksandr Piskun <oleksandr2088@icloud.com>

Author: kyteinsky

nc_py_api/_exceptions.py

@@ -1,6 +1,6 @@
 """Exceptions for the Nextcloud API."""
 
-from niquests import HTTPError, Response
+from niquests import AsyncResponse, HTTPError, Response
 
 
 class NextcloudException(Exception):
@@ -9,9 +9,15 @@ class NextcloudException(Exception):
     status_code: int
     reason: str
     info: str
-    response: Response | None
-
-    def __init__(self, status_code: int = 0, reason: str = "", info: str = "", response: Response | None = None):
+    response: AsyncResponse | Response | None
+
+    def __init__(
+        self,
+        status_code: int = 0,
+        reason: str = "",
+        info: str = "",
+        response: AsyncResponse | Response | None = None,
+    ):
         super(BaseException, self).__init__()
         self.status_code = status_code
         self.reason = reason
@@ -27,25 +33,25 @@ def __str__(self):
 class NextcloudExceptionNotModified(NextcloudException):
     """The exception indicates that there is no need to retransmit the requested resources."""
 
-    def __init__(self, reason="Not modified", info: str = "", response: Response | None = None):
+    def __init__(self, reason="Not modified", info: str = "", response: AsyncResponse | Response | None = None):
         super().__init__(304, reason=reason, info=info, response=response)
 
 
 class NextcloudExceptionNotFound(NextcloudException):
     """The exception that is thrown during operations when the object is not found."""
 
-    def __init__(self, reason="Not found", info: str = "", response: Response | None = None):
+    def __init__(self, reason="Not found", info: str = "", response: AsyncResponse | Response | None = None):
         super().__init__(404, reason=reason, info=info, response=response)
 
 
 class NextcloudMissingCapabilities(NextcloudException):
     """The exception that is thrown when required capability for API is missing."""
 
-    def __init__(self, reason="Missing capability", info: str = "", response: Response | None = None):
+    def __init__(self, reason="Missing capability", info: str = "", response: AsyncResponse | Response | None = None):
         super().__init__(412, reason=reason, info=info, response=response)
 
 
-def check_error(response: Response, info: str = ""):
+def check_error(response: AsyncResponse | Response, info: str = ""):
     """Checks HTTP code from Nextcloud, and raises exception in case of error.
 
     For the OCS and DAV `code` be code returned by HTTP and not the status from ``ocs_meta``.

pyproject.toml

@@ -48,7 +48,7 @@ dynamic = [
 dependencies = [
   "fastapi>=0.133",
   "filelock>=3.20.3,<4",
-  "niquests>=3,<4",
+  "niquests>=3.4.2,<4",
   "pydantic>=2.1.1",
   "python-dotenv>=1",
   "starlette>=1.0.1",