ayon-python-api 1.2.9__tar.gz → 1.2.11__tar.gz

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ayon_python_api
-Version: 1.2.9
+Version: 1.2.11
 Summary: AYON Python API
 Home-page: https://github.com/ynput/ayon-python-api
 Author: ynput.io

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/_api_helpers/bundles_addons.py

@@ -398,7 +398,7 @@ class BundlesAddonsAPI(BaseServerAPI):
 
         """
         response = self.upload_file(
-            "addons/install",
+            "api/addons/install",
             src_filepath,
             progress=progress,
             request_type=RequestTypes.post,
@@ -448,9 +448,11 @@ class BundlesAddonsAPI(BaseServerAPI):
             "private",
             filename
         )
-        url = f"{self.get_base_url()}/{endpoint}"
         self.download_file(
-            url, dst_filepath, chunk_size=chunk_size, progress=progress
+            endpoint,
+            dst_filepath,
+            chunk_size=chunk_size,
+            progress=progress,
         )
         return dst_filepath
 

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/_api_helpers/dependency_packages.py

@@ -189,7 +189,7 @@ class DependencyPackagesAPI(BaseServerAPI):
         route = self._get_dependency_package_route(src_filename)
         package_filepath = os.path.join(dst_directory, dst_filename)
         self.download_file(
-            route,
+            f"api/{route}",
             package_filepath,
             chunk_size=chunk_size,
             progress=progress
@@ -225,7 +225,7 @@ class DependencyPackagesAPI(BaseServerAPI):
             )
 
         route = self._get_dependency_package_route(dst_filename)
-        self.upload_file(route, src_filepath, progress=progress)
+        self.upload_file(f"api/{route}", src_filepath, progress=progress)
 
     def _get_dependency_package_route(
         self, filename: Optional[str] = None

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/_api_helpers/installers.py

@@ -143,7 +143,7 @@ class InstallersAPI(BaseServerAPI):
 
         """
         return self.download_file(
-            f"desktop/installers/{filename}",
+            f"api/desktop/installers/{filename}",
             dst_filepath,
             chunk_size=chunk_size,
             progress=progress
@@ -168,7 +168,7 @@ class InstallersAPI(BaseServerAPI):
 
         """
         return self.upload_file(
-            f"desktop/installers/{dst_filename}",
+            f"api/desktop/installers/{dst_filename}",
             src_filepath,
             progress=progress
         )

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/_api_helpers/links.py

@@ -203,6 +203,7 @@ class LinksAPI(BaseServerAPI):
         output_id: str,
         output_type: str,
         link_name: Optional[str] = None,
+        data: Optional[dict[str, Any]] = None,
     ) -> CreateLinkData:
         """Create link between 2 entities.
 
@@ -222,7 +223,8 @@ class LinksAPI(BaseServerAPI):
             output_id (str): Output entity id.
             output_type (str): Entity type of output entity.
             link_name (Optional[str]): Name of link.
-                Available from server version '1.0.0-rc.6'.
+            data (Optional[dict[str, Any]]): Additional data to be stored
+                with the link.
 
         Returns:
             CreateLinkData: Information about link.
@@ -242,6 +244,9 @@ class LinksAPI(BaseServerAPI):
         if link_name:
             kwargs["name"] = link_name
 
+        if data:
+            kwargs["data"] = data
+
         response = self.post(
             f"projects/{project_name}/links", **kwargs
         )

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/_api_helpers/thumbnails.py

@@ -256,7 +256,7 @@ class ThumbnailsAPI(BaseServerAPI):
 
         mime_type = get_media_mime_type(src_filepath)
         response = self.upload_file(
-            f"projects/{project_name}/thumbnails",
+            f"api/projects/{project_name}/thumbnails",
             src_filepath,
             request_type=RequestTypes.post,
             headers={"Content-Type": mime_type},
@@ -295,7 +295,7 @@ class ThumbnailsAPI(BaseServerAPI):
 
         mime_type = get_media_mime_type_for_stream(stream)
         response = self.upload_file_from_stream(
-            f"projects/{project_name}/thumbnails",
+            f"api/projects/{project_name}/thumbnails",
             stream,
             request_type=RequestTypes.post,
             headers={"Content-Type": mime_type},
@@ -325,7 +325,7 @@ class ThumbnailsAPI(BaseServerAPI):
 
         mime_type = get_media_mime_type(src_filepath)
         response = self.upload_file(
-            f"projects/{project_name}/thumbnails/{thumbnail_id}",
+            f"api/projects/{project_name}/thumbnails/{thumbnail_id}",
             src_filepath,
             request_type=RequestTypes.put,
             headers={"Content-Type": mime_type},
@@ -351,7 +351,7 @@ class ThumbnailsAPI(BaseServerAPI):
         """
         mime_type = get_media_mime_type_for_stream(stream)
         response = self.upload_file_from_stream(
-            f"projects/{project_name}/thumbnails/{thumbnail_id}",
+            f"api/projects/{project_name}/thumbnails/{thumbnail_id}",
             stream,
             request_type=RequestTypes.put,
             headers={"Content-Type": mime_type},

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/server_api.py

@@ -1352,7 +1352,7 @@ class ServerAPI(
 
     def _download_file_to_stream(
         self,
-        url: str,
+        endpoint: str,
         stream: StreamType,
         chunk_size: int,
         progress: TransferProgress,
@@ -1367,7 +1367,11 @@ class ServerAPI(
         else:
             get_func = self._session_functions_mapping[RequestTypes.get]
 
+        url = self._endpoint_to_url(endpoint, use_rest=False)
+        progress.set_source_url(url)
+
         retries = self.get_default_max_retries()
+        api_prepended = False
         for attempt in range(retries):
             # Continue in download
             offset = progress.get_transferred_size()
@@ -1376,6 +1380,18 @@ class ServerAPI(
 
             try:
                 with get_func(url, **kwargs) as response:
+                    # Auto-fix missing 'api/'
+                    if response.status_code == 405 and not api_prepended:
+                        api_prepended = True
+                        if (
+                            not endpoint.startswith(self._base_url)
+                            and not endpoint.startswith("api/")
+                        ):
+                            url = self._endpoint_to_url(
+                                endpoint, use_rest=True
+                            )
+                            progress.set_destination_url(url)
+                            continue
                     response.raise_for_status()
                     if progress.get_content_size() is None:
                         progress.set_content_size(
@@ -1395,6 +1411,12 @@ class ServerAPI(
                     raise
                 progress.next_attempt()
 
+        if api_prepended:
+            self.log.warning(
+                f"Auto-fixed endpoint '{endpoint}' -> 'api/{endpoint}'."
+                " Please fix the endpoit passed to the function."
+            )
+
     def download_file_to_stream(
         self,
         endpoint: str,
@@ -1427,17 +1449,14 @@ class ServerAPI(
         if not chunk_size:
             chunk_size = self.default_download_chunk_size
 
-        url = self._endpoint_to_url(endpoint, use_rest=False)
-
         if progress is None:
             progress = TransferProgress()
 
-        progress.set_source_url(url)
         progress.set_started()
 
         try:
             self._download_file_to_stream(
-                url, stream, chunk_size, progress
+                endpoint, stream, chunk_size, progress
             )
 
         except Exception as exc:
@@ -1585,13 +1604,7 @@ class ServerAPI(
             bytes: Chunk of file.
 
         """
-        # Get size of file
-        file_stream.seek(0, io.SEEK_END)
-        size = file_stream.tell()
         file_stream.seek(0)
-        # Set content size to progress object
-        progress.set_content_size(size)
-
         while True:
             chunk = file_stream.read(chunk_size)
             if not chunk:
@@ -1601,7 +1614,7 @@ class ServerAPI(
 
     def _upload_file(
         self,
-        url: str,
+        endpoint: str,
         stream: StreamType,
         progress: TransferProgress,
         request_type: Optional[RequestType] = None,
@@ -1611,7 +1624,7 @@ class ServerAPI(
         """Upload file to server.
 
         Args:
-            url (str): Url where file will be uploaded.
+            endpoint (str): Endpoint used to upload.
             stream (StreamType): File stream.
             progress (TransferProgress): Object that gives ability to track
                 progress.
@@ -1629,6 +1642,10 @@ class ServerAPI(
         if request_type is None:
             request_type = RequestTypes.put
 
+        endpoint = endpoint.lstrip("/")
+        url = self._endpoint_to_url(endpoint, use_rest=False)
+        progress.set_destination_url(url)
+
         if self._session is None:
             headers = kwargs.setdefault("headers", {})
             for key, value in self.get_headers().items():
@@ -1643,6 +1660,14 @@ class ServerAPI(
 
         retries = self.get_default_max_retries()
         response = None
+
+        # Get size of file
+        stream.seek(0, io.SEEK_END)
+        size = stream.tell()
+        # Set content size to progress object
+        progress.set_content_size(size)
+
+        api_prepended = False
         for attempt in range(retries):
             try:
                 response = post_func(
@@ -1652,6 +1677,16 @@ class ServerAPI(
                     ),
                     **kwargs
                 )
+                # Auto-fix missing 'api/'
+                if response.status_code == 405 and not api_prepended:
+                    api_prepended = True
+                    if (
+                        not endpoint.startswith(self._base_url)
+                        and not endpoint.startswith("api/")
+                    ):
+                        url = self._endpoint_to_url(endpoint, use_rest=True)
+                        progress.set_destination_url(url)
+                        continue
                 break
 
             except (
@@ -1664,6 +1699,11 @@ class ServerAPI(
                 progress.reset_transferred()
 
         response.raise_for_status()
+        if api_prepended:
+            self.log.warning(
+                f"Auto-fixed endpoint '{endpoint}' -> 'api/{endpoint}'."
+                " Please fix the endpoit passed to the function."
+            )
         return response
 
     def upload_file_from_stream(
@@ -1694,19 +1734,20 @@ class ServerAPI(
             requests.Response: Response object
 
         """
-        url = self._endpoint_to_url(endpoint)
-
         # Create dummy object so the function does not have to check
         #   'progress' variable everywhere
         if progress is None:
             progress = TransferProgress()
 
-        progress.set_destination_url(url)
         progress.set_started()
 
         try:
             return self._upload_file(
-                url, stream, progress, request_type, **kwargs
+                endpoint,
+                stream,
+                progress,
+                request_type,
+                **kwargs
             )
 
         except Exception as exc:
@@ -1751,7 +1792,11 @@ class ServerAPI(
 
         with open(filepath, "rb") as stream:
             return self.upload_file_from_stream(
-                endpoint, stream, progress, request_type, **kwargs
+                endpoint,
+                stream,
+                progress,
+                request_type,
+                **kwargs
             )
 
     def upload_reviewable(
@@ -1813,7 +1858,7 @@ class ServerAPI(
 
         query = prepare_query_string({"label": label or None})
         endpoint = (
-            f"/projects/{project_name}"
+            f"api/projects/{project_name}"
             f"/versions/{version_id}/reviewables{query}"
         )
         return self.upload_file(

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/utils.py

@@ -4,6 +4,8 @@ import os
 import re
 import datetime
 import copy
+import logging
+import json
 import uuid
 import string
 import platform
@@ -101,8 +103,9 @@ def _get_description(response):
     return HTTPStatus(response.status).description
 
 
-class RestApiResponse(object):
+class RestApiResponse:
     """API Response."""
+    log = logging.getLogger("RestApiResponse")
 
     def __init__(self, response, data=None):
         if response is None:
@@ -134,7 +137,7 @@ class RestApiResponse(object):
         if self._data is None:
             try:
                 self._data = self.orig_response.json()
-            except RequestsJSONDecodeError:
+            except (AttributeError, RequestsJSONDecodeError):
                 self._data = {}
         return self._data
 
@@ -178,6 +181,21 @@ class RestApiResponse(object):
         except requests.exceptions.HTTPError as exc:
             if message is None:
                 message = str(exc)
+
+            submsg = ""
+            if self.data:
+                submsg = json.dumps(self.data, indent=4)
+
+            self.log.warning(
+                "HTTP request error: %s%s%s",
+                message,
+                "\n" if submsg else "",
+                submsg,
+            )
+
+            detail = self.data.get("detail")
+            if detail:
+                message = f"{message} ({detail})"
             raise HTTPRequestError(message, exc.response)
 
     def __enter__(self, *args, **kwargs):

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_api/version.py

@@ -1,2 +1,2 @@
 """Package declaring Python API for AYON server."""
-__version__ = "1.2.9"
+__version__ = "1.2.11"

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/ayon_python_api.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ayon_python_api
-Version: 1.2.9
+Version: 1.2.11
 Summary: AYON Python API
 Home-page: https://github.com/ynput/ayon-python-api
 Author: ynput.io

{ayon_python_api-1.2.9 → ayon_python_api-1.2.11}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "ayon_python_api"
-version = "1.2.9"
+version = "1.2.11"
 description = "AYON Python API"
 license = {file = "LICENSE"}
 readme = {file = "README.md", content-type = "text/markdown"}