feat(api): fix account ID types

Change type of account_id path parameter from 'number' to 'string' for Cloudforce threat event and Magic Transit connector endpoints

Author: stainless-app[bot]

.stats.yml

@@ -1,4 +1,4 @@
 configured_endpoints: 1752
-openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/cloudflare%2Fcloudflare-630f05355fbbcf8b9ce657a1a5635aab2423b8387a5ffa7534173ffcdf57a305.yml
-openapi_spec_hash: ca3793864d354c1d505730523406f865
-config_hash: 135c41c52147f104907327e028f831e2
+openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/cloudflare%2Fcloudflare-dc675acbdcab1131f3681a184a8515deca17471c93595473e2474c4b5fafc7e2.yml
+openapi_spec_hash: 41b14e228d5ef99f8e1e09732ce2679c
+config_hash: 22b6cc50203041358f4b2b4e832b009d

src/cloudflare/resources/cloudforce_one/threat_events/attackers.py

@@ -42,7 +42,7 @@ def with_streaming_response(self) -> AttackersResourceWithStreamingResponse:
  def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -64,6 +64,8 @@ def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return self._get(
  f"/accounts/{account_id}/cloudforce-one/events/attackers",
  options=make_request_options(
@@ -96,7 +98,7 @@ def with_streaming_response(self) -> AsyncAttackersResourceWithStreamingResponse
  async def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -118,6 +120,8 @@ async def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return await self._get(
  f"/accounts/{account_id}/cloudforce-one/events/attackers",
  options=make_request_options(

src/cloudflare/resources/cloudforce_one/threat_events/categories.py

@@ -50,7 +50,7 @@ def with_streaming_response(self) -> CategoriesResourceWithStreamingResponse:
  def create(
  self,
  *,
- account_id: float,
+ account_id: str,
  kill_chain: float,
  name: str,
  mitre_attack: List[str] | NotGiven = NOT_GIVEN,
@@ -76,6 +76,8 @@ def create(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return self._post(
  f"/accounts/{account_id}/cloudforce-one/events/categories/create",
  body=maybe_transform(
@@ -96,7 +98,7 @@ def create(
  def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -118,6 +120,8 @@ def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return self._get(
  f"/accounts/{account_id}/cloudforce-one/events/categories",
  options=make_request_options(
@@ -130,7 +134,7 @@ def delete(
  self,
  category_id: str,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -154,6 +158,8 @@ def delete(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  if not category_id:
  raise ValueError(f"Expected a non-empty value for `category_id` but received {category_id!r}")
  return self._delete(
@@ -168,7 +174,7 @@ def edit(
  self,
  category_id: str,
  *,
- account_id: float,
+ account_id: str,
  kill_chain: float | NotGiven = NOT_GIVEN,
  mitre_attack: List[str] | NotGiven = NOT_GIVEN,
  name: str | NotGiven = NOT_GIVEN,
@@ -196,6 +202,8 @@ def edit(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  if not category_id:
  raise ValueError(f"Expected a non-empty value for `category_id` but received {category_id!r}")
  return self._patch(
@@ -219,7 +227,7 @@ def get(
  self,
  category_id: str,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -243,6 +251,8 @@ def get(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  if not category_id:
  raise ValueError(f"Expected a non-empty value for `category_id` but received {category_id!r}")
  return self._get(
@@ -277,7 +287,7 @@ def with_streaming_response(self) -> AsyncCategoriesResourceWithStreamingRespons
  async def create(
  self,
  *,
- account_id: float,
+ account_id: str,
  kill_chain: float,
  name: str,
  mitre_attack: List[str] | NotGiven = NOT_GIVEN,
@@ -303,6 +313,8 @@ async def create(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return await self._post(
  f"/accounts/{account_id}/cloudforce-one/events/categories/create",
  body=await async_maybe_transform(
@@ -323,7 +335,7 @@ async def create(
  async def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -345,6 +357,8 @@ async def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return await self._get(
  f"/accounts/{account_id}/cloudforce-one/events/categories",
  options=make_request_options(
@@ -357,7 +371,7 @@ async def delete(
  self,
  category_id: str,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -381,6 +395,8 @@ async def delete(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  if not category_id:
  raise ValueError(f"Expected a non-empty value for `category_id` but received {category_id!r}")
  return await self._delete(
@@ -395,7 +411,7 @@ async def edit(
  self,
  category_id: str,
  *,
- account_id: float,
+ account_id: str,
  kill_chain: float | NotGiven = NOT_GIVEN,
  mitre_attack: List[str] | NotGiven = NOT_GIVEN,
  name: str | NotGiven = NOT_GIVEN,
@@ -423,6 +439,8 @@ async def edit(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  if not category_id:
  raise ValueError(f"Expected a non-empty value for `category_id` but received {category_id!r}")
  return await self._patch(
@@ -446,7 +464,7 @@ async def get(
  self,
  category_id: str,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -470,6 +488,8 @@ async def get(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  if not category_id:
  raise ValueError(f"Expected a non-empty value for `category_id` but received {category_id!r}")
  return await self._get(

src/cloudflare/resources/cloudforce_one/threat_events/countries.py

@@ -42,7 +42,7 @@ def with_streaming_response(self) -> CountriesResourceWithStreamingResponse:
  def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -64,6 +64,8 @@ def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return self._get(
  f"/accounts/{account_id}/cloudforce-one/events/countries",
  options=make_request_options(
@@ -96,7 +98,7 @@ def with_streaming_response(self) -> AsyncCountriesResourceWithStreamingResponse
  async def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -118,6 +120,8 @@ async def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return await self._get(
  f"/accounts/{account_id}/cloudforce-one/events/countries",
  options=make_request_options(

src/cloudflare/resources/cloudforce_one/threat_events/crons.py

@@ -43,7 +43,7 @@ def with_streaming_response(self) -> CronsResourceWithStreamingResponse:
  def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -65,6 +65,8 @@ def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return self._get(
  f"/accounts/{account_id}/cloudforce-one/events/cron",
  options=make_request_options(
@@ -76,7 +78,7 @@ def list(
  def edit(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -98,6 +100,8 @@ def edit(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return self._patch(
  f"/accounts/{account_id}/cloudforce-one/events/cron",
  options=make_request_options(
@@ -130,7 +134,7 @@ def with_streaming_response(self) -> AsyncCronsResourceWithStreamingResponse:
  async def list(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -152,6 +156,8 @@ async def list(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return await self._get(
  f"/accounts/{account_id}/cloudforce-one/events/cron",
  options=make_request_options(
@@ -163,7 +169,7 @@ async def list(
  async def edit(
  self,
  *,
- account_id: float,
+ account_id: str,
  # Use the following arguments if you need to pass additional parameters to the API that aren't available via kwargs.
  # The extra values given here take precedence over values defined on the client or passed to this method.
  extra_headers: Headers | None = None,
@@ -185,6 +191,8 @@ async def edit(
 
  timeout: Override the client-level default timeout for this request, in seconds
  """
+ if not account_id:
+ raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
  return await self._patch(
  f"/accounts/{account_id}/cloudforce-one/events/cron",
  options=make_request_options(