zscaler-sdk-python 1.5.6__tar.gz → 1.5.8__tar.gz

{zscaler_sdk_python-1.5.6 → zscaler_sdk_python-1.5.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: zscaler-sdk-python
-Version: 1.5.6
+Version: 1.5.8
 Summary: Official Python SDK for the Zscaler Products
 Home-page: https://github.com/zscaler/zscaler-sdk-python
 License: MIT

{zscaler_sdk_python-1.5.6 → zscaler_sdk_python-1.5.8}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "zscaler-sdk-python"
-version = "1.5.6"
+version = "1.5.8"
 description = "Official Python SDK for the Zscaler Products"
 authors = ["Zscaler, Inc. <devrel@zscaler.com>"]
 license = "MIT"

{zscaler_sdk_python-1.5.6 → zscaler_sdk_python-1.5.8}/zscaler/__init__.py

@@ -29,7 +29,7 @@ __license__ = "MIT"
 __contributors__ = [
     "William Guilherme",
 ]
-__version__ = "1.5.6"
+__version__ = "1.5.8"
 
 
 from zscaler.oneapi_client import Client as ZscalerClient  # noqa

{zscaler_sdk_python-1.5.6 → zscaler_sdk_python-1.5.8}/zscaler/oneapi_response.py

@@ -68,14 +68,25 @@ class ZscalerAPIResponse:
             self._params["endTime"] = end_time
 
         # If service is ZPA and user did not specify a pagesize, set it to max=500 by default
-        if self._service_type == "ZPA" and "pagesize" not in self._params:
+        if self._service_type.upper() == "ZPA" and "pagesize" not in self._params:
             self._params["pagesize"] = self.SERVICE_PAGE_LIMITS["ZPA"]["max"]
 
         # If service is ZIA, ensure the pagination parameter is "pageSize" in camelCase
-        if self._service_type == "ZIA":
+        if self._service_type.upper() == "ZIA":
             if "page_size" in self._params:
                 self._params["pageSize"] = self._params.pop("page_size")
 
+        if self._service_type.upper() == "ZIA":
+            # If user supplied pageNumber, initialize self._page accordingly
+            if "pageNumber" in self._params:
+                try:
+                    self._page = int(self._params["pageNumber"])
+                except Exception:
+                    self._page = 1
+            # Always remove 'page' if present for ZIA
+            self._params.pop("page", None)
+            # logger.debug(f"[DEBUG] __init__ params after ZIA normalization: {self._params}")
+
         if res_details:
             content_type = res_details.headers.get("Content-Type", "").lower()
 
@@ -241,21 +252,25 @@ class ZscalerAPIResponse:
     #     return self._list, None
 
     def _fetch_next_page(self):
+        logger.debug(f"[DEBUG] _fetch_next_page called. service_type={self._service_type}, params={self._params}")
         if not self._has_next():
             logger.debug("No more pages to fetch")
             return [], None
 
-        if self._service_type == "ZDX":
+        if self._service_type.upper() == "ZDX":
+            logger.debug("[DEBUG] Taking ZDX pagination branch.")
             self._params["offset"] = self._next_offset
+        elif self._service_type.upper() == "ZIA":
+            logger.debug("[DEBUG] Taking ZIA pagination branch.")
+            self._page += 1
+            self._params["pageNumber"] = str(self._page)
+            # Ensure page parameter is never set for ZIA
+            self._params.pop("page", None)
+            logger.debug(f"[DEBUG] _fetch_next_page params for ZIA: {self._params}")
         else:
+            logger.debug("[DEBUG] Taking ZPA/other pagination branch.")
             self._page += 1
-
-            # ✅ Patch: support alternate pagination keys (e.g., Shadow IT)
-            if "pageNumber" in self._params:
-                self._params["pageNumber"] = str(self._page)
-                self._params.pop("page", None)  # Ensure no conflict
-            else:
-                self._params["page"] = self._page
+            self._params["page"] = self._page
 
         logger.debug(f"Requesting next page with params: {self._params}")
 

{zscaler_sdk_python-1.5.6 → zscaler_sdk_python-1.5.8}/zscaler/zia/shadow_it_report.py

@@ -184,7 +184,8 @@ class ShadowITAPI(APIClient):
             >>> if error:
             ...     print(f"Error updating applications: {error}")
             ...     return
-            ... print(f"Applications updated successfully: {updated_application.as_dict()}")
+            >>> if isinstance(updated_application, dict) and not updated_application:
+            ...     print("Applications updated successfully")
 
             Update the sanction state and custom tags of a cloud application:
 
@@ -195,7 +196,8 @@ class ShadowITAPI(APIClient):
             >>> if error:
             ...     print(f"Error updating applications: {error}")
             ...     return
-            ... print(f"Applications updated successfully: {updated_application.as_dict()}")
+            >>> if isinstance(updated_application, dict) and not updated_application:
+            ...     print("Applications updated successfully")
         """
         http_method = "put".upper()
         api_url = format_url(
@@ -236,11 +238,11 @@ class ShadowITAPI(APIClient):
 
         response, error = self._request_executor.execute(request)
 
-        try:
-            body = response.get_body()
-            result = self.form_response_body(body) if body else {}
-        except Exception as error:
+        if error:
             return (None, response, error)
+
+        body = response.get_body() if response else None
+        result = self.form_response_body(body) if body else {}
         return (result, response, None)
 
     def export_shadow_it_report(self, duration: str = "LAST_1_DAYS", **kwargs) -> tuple: