zscaler-sdk-python 1.9.7__tar.gz → 1.9.8__tar.gz

{zscaler_sdk_python-1.9.7 → zscaler_sdk_python-1.9.8}/PKG-INFO

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

{zscaler_sdk_python-1.9.7 → zscaler_sdk_python-1.9.8}/pyproject.toml

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

{zscaler_sdk_python-1.9.7 → zscaler_sdk_python-1.9.8}/zscaler/__init__.py

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

{zscaler_sdk_python-1.9.7 → zscaler_sdk_python-1.9.8}/zscaler/zpa/legacy.py

@@ -254,45 +254,68 @@ class LegacyZPAClientHelper:
         Returns:
             Tuple[requests.Response, dict]: Response object and request details.
         """
-        try:
-            base_url = f"{self.baseurl}{path}"
+        from time import sleep
+
+        base_url = f"{self.baseurl}{path}"
+        attempts = 0
+        max_retries = 5
 
-            headers = self.headers.copy()
-            headers.update(self.request_executor.get_custom_headers())
-            if not headers.get("Authorization"):
-                self.refreshToken()
-                # Re-copy headers after refreshToken() to include x-partner-id if it was added
+        while attempts < max_retries:
+            try:
                 headers = self.headers.copy()
                 headers.update(self.request_executor.get_custom_headers())
-                headers["Authorization"] = f"Bearer {self.access_token}"
-
-            response = requests.request(
-                method=method,
-                url=base_url,
-                headers=headers,
-                json=json,
-                params=params,
-                timeout=self.timeout,
-            )
-
-            _, err = check_response_for_error(base_url, response, response.text)
-            if err:
-                raise err
-
-            logger.info("Legacy client request executed successfully. "
-                        "Status: %d, URL: %s", response.status_code, base_url)
-
-            return response, {
-                "method": method,
-                "url": base_url,
-                "params": params or {},
-                "headers": headers,
-                "json": json or {},
-            }
-
-        except requests.RequestException as error:
-            logger.error(f"Error sending request: {error}")
-            raise ValueError(f"Request execution failed: {error}")
+                if not headers.get("Authorization"):
+                    self.refreshToken()
+                    # Re-copy headers after refreshToken() to include x-partner-id if it was added
+                    headers = self.headers.copy()
+                    headers.update(self.request_executor.get_custom_headers())
+                    headers["Authorization"] = f"Bearer {self.access_token}"
+
+                response = requests.request(
+                    method=method,
+                    url=base_url,
+                    headers=headers,
+                    json=json,
+                    params=params,
+                    timeout=self.timeout,
+                )
+
+                # Handle 429 rate limiting with retry-after header
+                # ZPA API returns lowercase 'retry-after' header
+                if response.status_code == 429:
+                    # Check both header variants (lowercase and uppercase)
+                    retry_after = response.headers.get("retry-after") or response.headers.get("Retry-After")
+                    sleep_time = int(retry_after) if retry_after else 2
+                    logger.warning(f"Rate limit exceeded (429). Retrying in {sleep_time} seconds. "
+                                   f"(Attempt {attempts + 1}/{max_retries})")
+                    sleep(sleep_time)
+                    attempts += 1
+                    continue
+
+                _, err = check_response_for_error(base_url, response, response.text)
+                if err:
+                    raise err
+
+                logger.info("Legacy client request executed successfully. "
+                            "Status: %d, URL: %s", response.status_code, base_url)
+
+                return response, {
+                    "method": method,
+                    "url": base_url,
+                    "params": params or {},
+                    "headers": headers,
+                    "json": json or {},
+                }
+
+            except requests.RequestException as error:
+                logger.error(f"Request to {base_url} failed: {error}")
+                if attempts == max_retries - 1:
+                    raise ValueError(f"Request execution failed: {error}")
+                logger.warning(f"Retrying... ({attempts + 1}/{max_retries})")
+                attempts += 1
+                sleep(5)
+
+        raise ValueError("Request execution failed after maximum retries.")
 
     def set_session(self, session: Any) -> None:
         """Dummy method for compatibility with the request executor."""

{zscaler_sdk_python-1.9.7 → zscaler_sdk_python-1.9.8}/zscaler/zpa/models/oauth2_user_code.py

@@ -14,8 +14,6 @@ ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
 OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 """
 
-# AUTO-GENERATED! DO NOT EDIT FILE DIRECTLY
-# SEE CONTRIBUTOR DOCUMENTATION
 from zscaler.oneapi_object import ZscalerObject
 from zscaler.oneapi_collection import ZscalerCollection
 

{zscaler_sdk_python-1.9.7 → zscaler_sdk_python-1.9.8}/zscaler/zpa/models/pra_portal.py

@@ -16,7 +16,7 @@ OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 
 from typing import Dict, List, Optional, Any, Union
 from zscaler.oneapi_object import ZscalerObject
-
+from zscaler.oneapi_collection import ZscalerCollection
 
 class PrivilegedRemoteAccessPortal(ZscalerObject):
     """
@@ -47,6 +47,9 @@ class PrivilegedRemoteAccessPortal(ZscalerObject):
             self.user_notification_enabled = config["userNotificationEnabled"] if "userNotificationEnabled" in config else None
             self.user_portal_gid = config["userPortalGid"] if "userPortalGid" in config else None
             self.user_portal_name = config["userPortalName"] if "userPortalName" in config else None
+            self.approval_reviewers = ZscalerCollection.form_list(
+                config["approvalReviewers"] if "approvalReviewers" in config else [], str
+            )
         else:
             self.id = None
             self.name = None
@@ -70,6 +73,7 @@ class PrivilegedRemoteAccessPortal(ZscalerObject):
             self.modified_time = None
             self.user_portal_gid = None
             self.user_portal_name = None
+            self.approval_reviewers = []
 
     def request_format(self) -> Dict[str, Any]:
         """
@@ -98,6 +102,7 @@ class PrivilegedRemoteAccessPortal(ZscalerObject):
             "userNotificationEnabled": self.user_notification_enabled,
             "userPortalGid": self.user_portal_gid,
             "userPortalName": self.user_portal_name,
+            "approvalReviewers": self.approval_reviewers,
         }
         parent_req_format.update(current_obj_format)
         return parent_req_format

{zscaler_sdk_python-1.9.7 → zscaler_sdk_python-1.9.8}/zscaler/zpa/pra_portal.py

@@ -145,6 +145,7 @@ class PRAPortalAPI(APIClient):
             certificate_id (str): The unique identifier of the certificate.
             domain (str): The domain of the PRA portal.
             enabled (bool): Whether the PRA portal is enabled (default is True).
+            approval_reviewers (list[str]): List of PRA Portal approval reviewers
 
         Returns:
             :obj:`Tuple`: PrivilegedRemoteAccessPortal: The newly created portal object.