etsy-python 1.2.2__tar.gz → 1.2.4__tar.gz

{etsy_python-1.2.2/etsy_python.egg-info → etsy_python-1.2.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: etsy-python
-Version: 1.2.2
+Version: 1.2.4
 Summary: Etsy API Client Library for Python
 Author: Amit Ray
 Author-email: mail@amitray.dev
@@ -181,7 +181,7 @@ token_data = oauth.get_access_token()
 ### 2. Initialize Client Session
 
 ```python
-from datetime import datetime
+from datetime import datetime, timedelta, timezone
 from etsy_python.v3.resources.Session import EtsyClient
 
 # Create client with tokens
@@ -189,7 +189,7 @@ client = EtsyClient(
     keystring="your_api_key",
     access_token=token_data["access_token"],
     refresh_token=token_data["refresh_token"],
-    expiry=datetime.utcnow() + timedelta(seconds=token_data["expires_in"])
+    expiry=datetime.now(tz=timezone.utc) + timedelta(seconds=token_data["expires_in"])
 )
 ```
 

{etsy_python-1.2.2 → etsy_python-1.2.4}/README.md

@@ -144,7 +144,7 @@ token_data = oauth.get_access_token()
 ### 2. Initialize Client Session
 
 ```python
-from datetime import datetime
+from datetime import datetime, timedelta, timezone
 from etsy_python.v3.resources.Session import EtsyClient
 
 # Create client with tokens
@@ -152,7 +152,7 @@ client = EtsyClient(
     keystring="your_api_key",
     access_token=token_data["access_token"],
     refresh_token=token_data["refresh_token"],
-    expiry=datetime.utcnow() + timedelta(seconds=token_data["expires_in"])
+    expiry=datetime.now(tz=timezone.utc) + timedelta(seconds=token_data["expires_in"])
 )
 ```
 

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/_version.py

@@ -1,3 +1,3 @@
 """Version information for etsy-python package."""
 
-__version__ = "1.2.2"
+__version__ = "1.2.4"

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/common/Request.py

@@ -1,4 +1,6 @@
 NO_RESPONSE_CODES = {204}
+# Kept for import compatibility only: EtsyClient treats every status >= 400 as
+# an error, not just these codes.
 ERROR_CODES = {400, 401, 403, 404, 409, 500, 503}
 
 DEFAULT_RESPONSE_MESSAGES = {

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/common/Utils.py

@@ -1,6 +1,6 @@
 import warnings
 from enum import Enum
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict, Iterable, List, Optional
 
 
 def warn_removed_legacy_param(operation_id: str) -> None:
@@ -25,6 +25,21 @@ def warn_removed_legacy_param(operation_id: str) -> None:
     )
 
 
+def join_ids(name: str, ids: Iterable[Any]) -> str:
+    """Join a required list of IDs into Etsy's comma-separated query value.
+
+    Raises ``TypeError`` for a string (iterating it would split the ID into
+    characters) and ``ValueError`` when no IDs are given, so a request Etsy
+    would reject is caught before it is sent.
+    """
+    if isinstance(ids, (str, bytes)):
+        raise TypeError(f"{name} must be a list of IDs, not a string")
+    joined = ",".join(map(str, ids)) if ids is not None else ""
+    if not joined:
+        raise ValueError(f"{name} is required and must contain at least one ID")
+    return joined
+
+
 def generate_get_uri(uri: str, params: Optional[Dict[str, Any]] = None) -> str:
     if not params:
         return uri

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/models/Listing.py

@@ -57,6 +57,28 @@ def _warn_if_personalization_used(
         )
 
 
+_FEATURED_RANK_DEPRECATION_MSG = (
+    "featured_rank was removed from updateListing by the Etsy API and is no "
+    "longer sent; it will be dropped from UpdateListingRequest in a future "
+    "major version."
+)
+
+
+def _warn_if_featured_rank_used(featured_rank: Optional[int]) -> None:
+    """Emit DeprecationWarning when the removed featured_rank field is set.
+
+    ``0`` does not warn: it is the default a fetched listing reports, so
+    callers that round-trip a listing into an update would otherwise warn on
+    every call. This matches the personalization fields, which skip falsy
+    values. ``stacklevel=3`` points at the caller of the model ``__init__`` or
+    the ``featured_rank`` setter.
+    """
+    if featured_rank:
+        warnings.warn(
+            _FEATURED_RANK_DEPRECATION_MSG, DeprecationWarning, stacklevel=3
+        )
+
+
 class _PersonalizationFieldsMixin:
     """Accessors for the personalization fields Etsy removed from listings.
 
@@ -310,7 +332,6 @@ class UpdateListingRequest(_PersonalizationFieldsMixin, _ECGTFieldsMixin, Reques
         "item_weight_unit",
         "item_dimensions_unit",
         "tags",
-        "featured_rank",
         "production_partner_ids",
         "_type",
         *ECGT_FIELDS,
@@ -375,7 +396,10 @@ class UpdateListingRequest(_PersonalizationFieldsMixin, _ECGTFieldsMixin, Reques
         self.tags = tags
         self.who_made = who_made
         self.when_made = when_made
-        self.featured_rank = featured_rank
+        # Removed from the updateListing body by Etsy: stored under an
+        # underscore name so todict never sends it.
+        self._featured_rank = featured_rank
+        _warn_if_featured_rank_used(featured_rank)
         self._store_personalization(
             is_personalizable,
             personalization_is_required,
@@ -400,6 +424,15 @@ class UpdateListingRequest(_PersonalizationFieldsMixin, _ECGTFieldsMixin, Reques
             mandatory=UpdateListingRequest.mandatory,
         )
 
+    @property
+    def featured_rank(self) -> Optional[int]:
+        return self._featured_rank
+
+    @featured_rank.setter
+    def featured_rank(self, value: Optional[int]) -> None:
+        self._featured_rank = value
+        _warn_if_featured_rank_used(value)
+
 
 class UpdateListingInventoryRequest(Request):
     nullable: List[str] = ["readiness_state_on_property"]

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/resources/Listing.py

@@ -2,7 +2,7 @@ import warnings
 from dataclasses import dataclass
 from typing import Optional, List, Dict, Any, Union
 
-from etsy_python.v3.common.Utils import warn_removed_legacy_param
+from etsy_python.v3.common.Utils import join_ids, warn_removed_legacy_param
 from etsy_python.v3.enums.Listing import Includes, State, SortOn, SortOrder
 from etsy_python.v3.exceptions.RequestException import RequestException
 from etsy_python.v3.models.Listing import (
@@ -150,7 +150,7 @@ class ListingResource:
     ) -> Union[Response, RequestException]:
         endpoint = "/listings/batch"
         query_params: Dict[str, Any] = {
-            "listing_ids": ",".join(list(map(str, listing_ids))),
+            "listing_ids": join_ids("listing_ids", listing_ids),
             "includes": ",".join(list(map(lambda inc: inc.value, includes)))
             if includes is not None
             else None,
@@ -179,7 +179,7 @@ class ListingResource:
     ) -> Union[Response, RequestException]:
         endpoint = "/listings/batch/inventory"
         query_params: Dict[str, Any] = {
-            "listing_ids": ",".join(list(map(str, listing_ids))),
+            "listing_ids": join_ids("listing_ids", listing_ids),
         }
         return self.session.make_request(endpoint, query_params=query_params)
 
@@ -188,7 +188,7 @@ class ListingResource:
     ) -> Union[Response, RequestException]:
         endpoint = "/listings/batch/shipping"
         query_params: Dict[str, Any] = {
-            "listing_ids": ",".join(list(map(str, listing_ids))),
+            "listing_ids": join_ids("listing_ids", listing_ids),
         }
         return self.session.make_request(endpoint, query_params=query_params)
 
@@ -260,18 +260,21 @@ class ListingResource:
     def get_listings_by_shop_section_id(
         self,
         shop_id: int,
-        shop_section_ids: Optional[List[int]] = None,
+        shop_section_ids: List[int],
         limit: int = 25,
         offset: int = 0,
         sort_on: SortOn = SortOn.CREATED,
         sort_order: SortOrder = SortOrder.DESC,
         legacy: Optional[bool] = None,
     ) -> Union[Response, RequestException]:
+        """Get listings in the given shop sections.
+
+        Raises ``ValueError`` if ``shop_section_ids`` is empty and
+        ``TypeError`` if it is a string, before any request is sent.
+        """
         endpoint = f"/shops/{shop_id}/shop-sections/listings"
         query_params: Dict[str, Any] = {
-            "shop_section_ids": ",".join(list(map(str, shop_section_ids)))
-            if shop_section_ids is not None
-            else None,
+            "shop_section_ids": join_ids("shop_section_ids", shop_section_ids),
             "limit": limit,
             "offset": offset,
             "sort_on": sort_on.value,

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/resources/ListingInventory.py

@@ -35,6 +35,7 @@ class ListingInventoryResource:
         self, listing_id: int, listing_inventory: UpdateListingInventoryRequest,
         legacy: Optional[bool] = None,
         max_variations_supported: Optional[MaxVariationsSupported] = None,
+        enable_scale_validation: Optional[bool] = None,
     ) -> Union[Response, RequestException]:
         if legacy is not None:
             warn_removed_legacy_param("updateListingInventory")
@@ -43,6 +44,7 @@ class ListingInventoryResource:
             "max_variations_supported": max_variations_supported.value
             if max_variations_supported is not None
             else None,
+            "enable_scale_validation": enable_scale_validation,
         }
         return self.session.make_request(
             endpoint, method=Method.PUT, payload=listing_inventory,

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/resources/ListingVideo.py

@@ -1,3 +1,4 @@
+import warnings
 from dataclasses import dataclass
 from typing import Optional, Union
 
@@ -37,14 +38,22 @@ class ListingVideoResource:
     ) -> Union[Response, RequestException]:
         """Upload a video to a listing.
 
-        Pass ``is_multi_video=True`` to keep existing videos on the listing.
-        Omitting it preserves the former single-video behaviour, where the
-        upload replaces any video already attached.
+        ``is_multi_video`` is deprecated: Etsy removed it from
+        uploadListingVideo, so any value is discarded and not sent, and Etsy
+        no longer documents whether an upload replaces an existing video. To
+        replace one, remove it with ``delete_listing_video`` before uploading
+        the new video. Etsy limits uploads to 10 per listing per day.
         """
+        if is_multi_video is not None:
+            warnings.warn(
+                "'is_multi_video' was removed from uploadListingVideo by Etsy "
+                "and is no longer sent; to replace a video, remove it with "
+                "delete_listing_video first. The argument will be dropped from "
+                "this method in a future major version.",
+                DeprecationWarning,
+                stacklevel=2,
+            )
         endpoint = f"/shops/{shop_id}/listings/{listing_id}/videos"
         return self.session.make_request(
-            endpoint,
-            method=Method.POST,
-            payload=listing_video,
-            query_params={"is_multi_video": is_multi_video},
+            endpoint, method=Method.POST, payload=listing_video
         )

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/resources/Payment.py

@@ -2,6 +2,7 @@ import warnings
 from dataclasses import dataclass
 from typing import Union, Dict, Any, List
 
+from etsy_python.v3.common.Utils import join_ids
 from etsy_python.v3.exceptions.RequestException import RequestException
 from etsy_python.v3.resources.Session import EtsyClient
 from etsy_python.v3.resources.Response import Response
@@ -16,7 +17,7 @@ class PaymentResource:
     ) -> Union[Response, RequestException]:
         endpoint = f"/shops/{shop_id}/payment-account/ledger-entries/payments"
         query_params: Dict[str, Any] = {
-            "ledger_entry_ids": ",".join(list(map(str, ledger_entry_ids)))
+            "ledger_entry_ids": join_ids("ledger_entry_ids", ledger_entry_ids)
         }
         return self.session.make_request(endpoint, query_params=query_params)
 
@@ -42,5 +43,5 @@ class PaymentResource:
         self, shop_id: int, payment_ids: List[int]
     ) -> Union[Response, RequestException]:
         endpoint = f"/shops/{shop_id}/payments"
-        query_params: Dict[str, Any] = {"payment_ids": ",".join(list(map(str, payment_ids)))}
+        query_params: Dict[str, Any] = {"payment_ids": join_ids("payment_ids", payment_ids)}
         return self.session.make_request(endpoint, query_params=query_params)

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/resources/Session.py

@@ -3,7 +3,7 @@ from typing import Any, Callable, Dict, Optional
 
 from requests import Session
 
-from etsy_python.v3.common.Request import ERROR_CODES, NO_RESPONSE_CODES
+from etsy_python.v3.common.Request import NO_RESPONSE_CODES
 from etsy_python.v3.common.Env import environment
 from etsy_python.v3.common.Utils import generate_get_uri
 from etsy_python.v3.resources.enums.Request import Method
@@ -35,22 +35,51 @@ class EtsyClient:
         self.session.headers = self._get_resource_headers(keystring, access_token)
 
     def update_token(self) -> tuple:
-        self._remove_authorization_header()
         refresh_json = self._get_refresh_json(self.keystring, self.refresh_token)
-        response = self.session.post(environment.token_url, json=refresh_json)
+        # Drop Authorization for this call only. The shared session header is
+        # never touched, so a failed refresh cannot leave later requests
+        # unauthenticated.
+        response = self.session.post(
+            environment.token_url, json=refresh_json, headers={"Authorization": None}
+        )
         response_json = self._process_request(response).message
-
-        self.access_token = response_json.get("access_token")
-        self.refresh_token = response_json.get("refresh_token")
-        if not (self.access_token and self.refresh_token):
+        if not isinstance(response_json, dict):
+            raise RequestException(
+                code=response.status_code,
+                error="Token refresh returned a non-JSON body",
+            )
+        access_token = response_json.get("access_token")
+        refresh_token = response_json.get("refresh_token")
+        expires_in = response_json.get("expires_in")
+        # expires_in must be a positive int; bool is an int subclass, and
+        # floats allow NaN/inf, which timedelta cannot handle.
+        invalid = [
+            name
+            for name, ok in (
+                ("access_token", bool(access_token)),
+                ("refresh_token", bool(refresh_token)),
+                (
+                    "expires_in",
+                    isinstance(expires_in, int)
+                    and not isinstance(expires_in, bool)
+                    and 0 < expires_in <= 10**9,
+                ),
+            )
+            if not ok
+        ]
+        if invalid:
             raise RequestException(
-                code=401, error=response_json.get("error", "Something went wrong!")
+                code=401,
+                error=response_json.get("error", "Something went wrong!"),
+                error_description=response_json.get("error_description")
+                or "Token refresh response has missing or invalid: "
+                + ", ".join(invalid),
             )
 
-        updated_expiry = datetime.utcnow() + timedelta(
-            seconds=response_json["expires_in"]
-        )
-        self.expiry = updated_expiry
+        # Store the new tokens only once the whole response is valid.
+        self.access_token = access_token
+        self.refresh_token = refresh_token
+        self.expiry = datetime.now(tz=timezone.utc) + timedelta(seconds=expires_in)
 
         self.session.headers.update(
             self._prepare_authorization_token(self.access_token)
@@ -70,9 +99,6 @@ class EtsyClient:
     def _prepare_authorization_token(access_token: str) -> dict:
         return {"Authorization": f"Bearer {access_token}"}
 
-    def _remove_authorization_header(self) -> None:
-        self.session.headers.pop("Authorization", None)
-
     @staticmethod
     def _get_request_headers(keystring: str) -> dict:
         return {"Accept": "application/json", "x-api-key": keystring}
@@ -132,21 +158,26 @@ class EtsyClient:
         return self._process_request(response)
 
     def _process_request(self, response: Any) -> Any:
-        is_error = response.status_code in ERROR_CODES
+        # Any 4xx/5xx is an error, including codes Etsy documents later
+        # (e.g. 429 for the uploadListingVideo daily limit).
+        is_error = response.status_code >= 400
 
         rate_limits = None
         if "X-Limit-Per-Day" in response.headers:
             rate_limits = RateLimit(
-                response.headers["X-Limit-Per-Day"],
-                response.headers["X-Remaining-This-Second"],
-                response.headers["X-Limit-Per-Day"],
-                response.headers["X-Remaining-Today"],
+                response.headers.get("X-Limit-Per-Second"),
+                response.headers.get("X-Remaining-This-Second"),
+                response.headers.get("X-Limit-Per-Day"),
+                response.headers.get("X-Remaining-Today"),
             )
 
-        response_json = (
-            response.json() if response.status_code not in NO_RESPONSE_CODES else None
-        )
         if is_error:
+            try:
+                response_json = response.json()
+            except ValueError:
+                # Gateway and method errors (405, 502, 504) can return a
+                # non-JSON or empty body; keep the text, or the reason phrase.
+                response_json = response.text or response.reason
             error_response = (
                 response_json
                 if isinstance(response_json, dict)
@@ -160,6 +191,13 @@ class EtsyClient:
                 error_description,
                 rate_limits=rate_limits,
             )
+        response_json = None
+        if response.status_code not in NO_RESPONSE_CODES:
+            try:
+                response_json = response.json()
+            except ValueError:
+                # A success with an empty or non-JSON body (202, 304, ...).
+                response_json = response.text
         return Response(
             response.status_code, response_json or "OK", rate_limits=rate_limits
         )

{etsy_python-1.2.2 → etsy_python-1.2.4}/etsy_python/v3/resources/UserAddress.py

@@ -1,3 +1,4 @@
+import warnings
 from dataclasses import dataclass
 from typing import Union, Dict, Any
 
@@ -14,6 +15,14 @@ class UserAddressResource:
     def delete_user_address(
         self, user_address_id: int
     ) -> Union[Response, RequestException]:
+        """Deprecated: Etsy removed deleteUserAddress from the Open API v3 spec."""
+        warnings.warn(
+            "delete_user_address is deprecated: Etsy removed deleteUserAddress "
+            "from the Open API v3 spec, so the request may fail. It will be "
+            "removed in a future major version.",
+            DeprecationWarning,
+            stacklevel=2,
+        )
         endpoint = f"/user/addresses/{user_address_id}"
         return self.session.make_request(endpoint, method=Method.DELETE)
 

{etsy_python-1.2.2 → etsy_python-1.2.4/etsy_python.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: etsy-python
-Version: 1.2.2
+Version: 1.2.4
 Summary: Etsy API Client Library for Python
 Author: Amit Ray
 Author-email: mail@amitray.dev
@@ -181,7 +181,7 @@ token_data = oauth.get_access_token()
 ### 2. Initialize Client Session
 
 ```python
-from datetime import datetime
+from datetime import datetime, timedelta, timezone
 from etsy_python.v3.resources.Session import EtsyClient
 
 # Create client with tokens
@@ -189,7 +189,7 @@ client = EtsyClient(
     keystring="your_api_key",
     access_token=token_data["access_token"],
     refresh_token=token_data["refresh_token"],
-    expiry=datetime.utcnow() + timedelta(seconds=token_data["expires_in"])
+    expiry=datetime.now(tz=timezone.utc) + timedelta(seconds=token_data["expires_in"])
 )
 ```