threecommon 0.7.0__tar.gz → 0.7.1__tar.gz

{threecommon-0.7.0 → threecommon-0.7.1}/CHANGELOG.md

@@ -5,6 +5,27 @@ versions follow [SemVer](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
 
+## 0.7.1
+
+### Fixed
+
+- Requests without a body no longer send `Content-Type: application/json`.
+  `DELETE` (e.g. `invoices.delete_draft`, `contacts.delete`) and the
+  action-style POST methods previously advertised a JSON body, which servers
+  enforcing `Content-Type` against an empty body reject with HTTP 400.
+  `build_headers` now sets `Content-Type` only when a body is present. Applies
+  to both the sync and async clients.
+
+### Changed
+
+- The action-style POST methods (`invoices.finalize`, `invoices.auto_charge`,
+  `features.archive`, `features.unarchive`, `prices.archive`,
+  `prices.unarchive`, `subscriptions.activate`, `subscriptions.mark_unpaid`,
+  `subscriptions.bill`, `subscriptions.renew`) no longer send an empty `{}`
+  request body; these endpoints take no body per the API spec. `void`,
+  `cancel`, and `cancel_immediately`, which accept an optional body, are
+  unchanged.
+
 ## 0.7.0
 
 ### Added

{threecommon-0.7.0 → threecommon-0.7.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: threecommon
-Version: 0.7.0
+Version: 0.7.1
 Summary: Official Python client for the 3Common Public API.
 Project-URL: Homepage, https://github.com/3-Common/sdk/tree/main/sdk-python
 Project-URL: Issues, https://github.com/3-Common/sdk/issues

{threecommon-0.7.0 → threecommon-0.7.1}/pyproject.toml

@@ -10,7 +10,7 @@ license = { text = "MIT" }
 authors = [{ name = "3Common", email = "support@3common.com" }]
 keywords = ["3common", "sdk", "api-client", "events", "invoices"]
 requires-python = ">=3.10"
-version = "0.7.0"
+version = "0.7.1"
 dependencies = [
     "httpx>=0.27,<1.0",
     "pydantic>=2.7,<3.0",

{threecommon-0.7.0 → threecommon-0.7.1}/src/threecommon/_core/headers.py

@@ -25,6 +25,7 @@ def build_headers(
     user_agent_extra: str | None = None,
     telemetry_header: str | None = None,
     idempotency_key: str | None = None,
+    has_body: bool = True,
 ) -> dict[str, str]:
     """Return a fresh header dict populated with every header the SDK sends."""
     headers: dict[str, str] = {
@@ -32,8 +33,12 @@ def build_headers(
         "Threecommon-Version": api_version,
         "User-Agent": f"ThreeCommonPython/{sdk_version} ({user_agent_suffix(user_agent_extra)})",
         "Accept": "application/json",
-        "Content-Type": "application/json",
     }
+    # Bodyless requests (DELETE, action-style POSTs like finalize/auto_charge)
+    # must not advertise a JSON body: a server enforcing Content-Type against
+    # an empty body rejects them with a 400.
+    if has_body:
+        headers["Content-Type"] = "application/json"
     if telemetry_header:
         headers["Threecommon-Client-Telemetry"] = telemetry_header
     if idempotency_key:

{threecommon-0.7.0 → threecommon-0.7.1}/src/threecommon/_core/http_client.py

@@ -172,6 +172,7 @@ def _build_request_headers(
     telemetry: Telemetry,
     idempotency_key: str | None,
     user_agent_extra: str | None,
+    has_body: bool,
 ) -> dict[str, str]:
     return build_headers(
         api_key=api_key,
@@ -180,6 +181,7 @@ def _build_request_headers(
         user_agent_extra=user_agent_extra,
         telemetry_header=telemetry.header_value(sdk_version=VERSION, api_version=api_version),
         idempotency_key=idempotency_key,
+        has_body=has_body,
     )
 
 
@@ -245,6 +247,7 @@ class HTTPClient:
                 telemetry=self._opts.telemetry,
                 idempotency_key=resolved.idempotency_key,
                 user_agent_extra=self._opts.user_agent_extra,
+                has_body=resolved.body is not None,
             )
 
             start = time.monotonic()
@@ -352,6 +355,7 @@ class AsyncHTTPClient:
                 telemetry=self._opts.telemetry,
                 idempotency_key=resolved.idempotency_key,
                 user_agent_extra=self._opts.user_agent_extra,
+                has_body=resolved.body is not None,
             )
 
             start = time.monotonic()

{threecommon-0.7.0 → threecommon-0.7.1}/src/threecommon/features/service.py

@@ -133,7 +133,7 @@ class FeaturesService:
         """Soft-archive a feature. Idempotent."""
         _require_id("archive", feature_id)
         response = self._http.request(
-            Request(method="POST", path=f"{_path_for(feature_id)}/archive", body={})
+            Request(method="POST", path=f"{_path_for(feature_id)}/archive")
         )
         return Feature.model_validate(response["data"])
 
@@ -141,7 +141,7 @@ class FeaturesService:
         """Reactivate a previously archived feature. Idempotent."""
         _require_id("unarchive", feature_id)
         response = self._http.request(
-            Request(method="POST", path=f"{_path_for(feature_id)}/unarchive", body={})
+            Request(method="POST", path=f"{_path_for(feature_id)}/unarchive")
         )
         return Feature.model_validate(response["data"])
 
@@ -220,14 +220,14 @@ class AsyncFeaturesService:
     async def archive(self, feature_id: str) -> Feature:
         _require_id("archive", feature_id)
         response = await self._http.request(
-            Request(method="POST", path=f"{_path_for(feature_id)}/archive", body={})
+            Request(method="POST", path=f"{_path_for(feature_id)}/archive")
         )
         return Feature.model_validate(response["data"])
 
     async def unarchive(self, feature_id: str) -> Feature:
         _require_id("unarchive", feature_id)
         response = await self._http.request(
-            Request(method="POST", path=f"{_path_for(feature_id)}/unarchive", body={})
+            Request(method="POST", path=f"{_path_for(feature_id)}/unarchive")
         )
         return Feature.model_validate(response["data"])
 

{threecommon-0.7.0 → threecommon-0.7.1}/src/threecommon/invoices/service.py

@@ -138,7 +138,7 @@ class InvoicesService:
         """Finalize a draft invoice: assign a number, stamp ``issuedAt``, set status ``open``."""
         _require_id("finalize", invoice_id)
         response = self._http.request(
-            Request(method="POST", path=_action_path(invoice_id, "finalize"), body={})
+            Request(method="POST", path=_action_path(invoice_id, "finalize"))
         )
         return Invoice.model_validate(response["data"])
 
@@ -177,7 +177,7 @@ class InvoicesService:
         """
         _require_id("auto_charge", invoice_id)
         response = self._http.request(
-            Request(method="POST", path=_action_path(invoice_id, "auto_charge"), body={})
+            Request(method="POST", path=_action_path(invoice_id, "auto_charge"))
         )
         return _build_auto_charge_result(response)
 
@@ -283,7 +283,7 @@ class AsyncInvoicesService:
     async def finalize(self, invoice_id: str) -> Invoice:
         _require_id("finalize", invoice_id)
         response = await self._http.request(
-            Request(method="POST", path=_action_path(invoice_id, "finalize"), body={})
+            Request(method="POST", path=_action_path(invoice_id, "finalize"))
         )
         return Invoice.model_validate(response["data"])
 
@@ -311,7 +311,7 @@ class AsyncInvoicesService:
     async def auto_charge(self, invoice_id: str) -> AutoChargeResult:
         _require_id("auto_charge", invoice_id)
         response = await self._http.request(
-            Request(method="POST", path=_action_path(invoice_id, "auto_charge"), body={})
+            Request(method="POST", path=_action_path(invoice_id, "auto_charge"))
         )
         return _build_auto_charge_result(response)
 

{threecommon-0.7.0 → threecommon-0.7.1}/src/threecommon/prices/service.py

@@ -114,16 +114,14 @@ class PricesService:
     def archive(self, price_id: str) -> Price:
         """Soft-archive a price. Idempotent."""
         _require_id("archive", price_id)
-        response = self._http.request(
-            Request(method="POST", path=f"{_path_for(price_id)}/archive", body={})
-        )
+        response = self._http.request(Request(method="POST", path=f"{_path_for(price_id)}/archive"))
         return Price.model_validate(response["data"])
 
     def unarchive(self, price_id: str) -> Price:
         """Reactivate a previously archived price. Idempotent."""
         _require_id("unarchive", price_id)
         response = self._http.request(
-            Request(method="POST", path=f"{_path_for(price_id)}/unarchive", body={})
+            Request(method="POST", path=f"{_path_for(price_id)}/unarchive")
         )
         return Price.model_validate(response["data"])
 
@@ -196,14 +194,14 @@ class AsyncPricesService:
     async def archive(self, price_id: str) -> Price:
         _require_id("archive", price_id)
         response = await self._http.request(
-            Request(method="POST", path=f"{_path_for(price_id)}/archive", body={})
+            Request(method="POST", path=f"{_path_for(price_id)}/archive")
         )
         return Price.model_validate(response["data"])
 
     async def unarchive(self, price_id: str) -> Price:
         _require_id("unarchive", price_id)
         response = await self._http.request(
-            Request(method="POST", path=f"{_path_for(price_id)}/unarchive", body={})
+            Request(method="POST", path=f"{_path_for(price_id)}/unarchive")
         )
         return Price.model_validate(response["data"])
 

{threecommon-0.7.0 → threecommon-0.7.1}/src/threecommon/subscriptions/service.py

@@ -146,7 +146,7 @@ class SubscriptionsService:
         """Transition an incomplete or trialing subscription to ``active``."""
         _require_id("activate", subscription_id)
         response = self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "activate"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "activate"))
         )
         return Subscription.model_validate(response["data"])
 
@@ -178,7 +178,7 @@ class SubscriptionsService:
         """Admin override — mark a subscription ``unpaid`` (terminal)."""
         _require_id("mark_unpaid", subscription_id)
         response = self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "mark-unpaid"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "mark-unpaid"))
         )
         return Subscription.model_validate(response["data"])
 
@@ -186,7 +186,7 @@ class SubscriptionsService:
         """Generate a draft invoice for the current period without advancing it."""
         _require_id("bill", subscription_id)
         response = self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "bill"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "bill"))
         )
         return _build_bill_result(response)
 
@@ -194,7 +194,7 @@ class SubscriptionsService:
         """Advance the subscription to its next billing period and generate an invoice."""
         _require_id("renew", subscription_id)
         response = self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "renew"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "renew"))
         )
         return _build_renew_result(response)
 
@@ -293,7 +293,7 @@ class AsyncSubscriptionsService:
     async def activate(self, subscription_id: str) -> Subscription:
         _require_id("activate", subscription_id)
         response = await self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "activate"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "activate"))
         )
         return Subscription.model_validate(response["data"])
 
@@ -322,21 +322,21 @@ class AsyncSubscriptionsService:
     async def mark_unpaid(self, subscription_id: str) -> Subscription:
         _require_id("mark_unpaid", subscription_id)
         response = await self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "mark-unpaid"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "mark-unpaid"))
         )
         return Subscription.model_validate(response["data"])
 
     async def bill(self, subscription_id: str) -> BillSubscriptionResult:
         _require_id("bill", subscription_id)
         response = await self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "bill"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "bill"))
         )
         return _build_bill_result(response)
 
     async def renew(self, subscription_id: str) -> RenewSubscriptionResult:
         _require_id("renew", subscription_id)
         response = await self._http.request(
-            Request(method="POST", path=_action_path(subscription_id, "renew"), body={})
+            Request(method="POST", path=_action_path(subscription_id, "renew"))
         )
         return _build_renew_result(response)