Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support for APIs in the new API version 2024-09-30.acacia #1458

Merged
merged 7 commits into from
Oct 1, 2024

Conversation

ramya-stripe
Copy link
Contributor

@ramya-stripe ramya-stripe commented Sep 27, 2024

Changelog

This release changes the pinned API version to 2024-09-30.acacia. Please read the API Upgrade Guide and carefully review the API changes before upgrading.

⚠️ Breaking changes

Please refer to our migration guide for v13 for more information about the backwards incompatible changes.

StripeClient and related changes

  • Move StripeClient and requestor logic to APIRequestor.
    • StripeClient#request is still available, but is deprecated and will be removed. We encourage StripeClient#raw_request as a replacement (see other breaking changes for more detail).
  • Repurpose and introduce StripeClient as the the entry-point to the service-based pattern, a new interface for calling the Stripe API with many benefits over the existing resource-based paradigm. Services are available under the v1 and v2 accessors.
    • No global config: you can simultaneously use multiple clients with different configuration options (such as API keys)
    • No extra API calls. All API endpoints can be accessed with a single method call. You don't have to call retrieve before doing an update.
    • No static methods. Much easier mocking.

Other breaking changes

  • Adjust default values around retries for HTTP requests. You can use the old defaults by setting them explicitly. New values are:
    • max retries: 0 -> 2
    • max retry delay (seconds) 2 -> 5
  • Remove StripeClient#connection_manager. This was a legacy method from years ago.
  • Singleton retrieve method now requires params to be passed as the first argument. Existing calls to singleton retrieve method with only opts argument will have to be updated to account for the addition of params argument.
    params = { expand: ["available"] }
    opts = { stripe_account: "acct_123" }
    
    # ❌ No longer works
    Stripe::Balance.retrieve(opts)
    
    # ✅ Correct way to call retrieve method
    Stripe::Balance.retrieve(params, opts)
  • Moved the Stripe.raw_request() method that was recently added to StripeClient. This will use the configuration set on the StripeClient instead of the global configuration used before.
  • Remove APIResource.request. Instead, use StripeClient#raw_request now.
    # Instead of
    Stripe::APIResource.request(:get, "/v1/endpoint", params, opts)
    
    # do
    client = Stripe::StripeClient.new(...)
    resp = client.raw_request(:get, "/v1/endpoint", params: params, opts: opts)
  • Add an additional parameter to APIResource.execute_resource_request. However, we discourage use of this in favor of StripeClient#raw_request.
    APIResource.execute_resource_request(method, url, params = {}, opts = {}, usage = [])
    # is now, with base_address being one of [:api, :files, :connect, :meter_events]
    APIResource.execute_resource_request(method, url, base_address = :api, params = {}, opts = {}, usage = [])
  • Change parameters to APIRequestor.execute_request (previously StripeClient.execute_request). It now returns all request options from our internal request framework as the second value in the returned tuple, instead of only the API key used:
    # Before
    obj, api_key = StripeClient.execute_request(method, path, api_base: nil,
                                                api_key: nil, headers: {}, params: {}, usage: [])
    
    # is now, with base_address being one of [:api, :files, :connect, :meter_events]
    
    obj, opts = APIRequestor.execute_request(method, path, base_address,
                                            params: {}, opts: {}, usage: [])
    puts(opts) # will output {api_key: "sk_test_123", stripe_account: "acct_123"}

Additions

  • Add support for new Usage Billing APIs Billing.MeterEvent, Billing.MeterEventAdjustments, Billing.MeterEventSession, Billing.MeterEventStream and the new Events API Core.Events in the v2 namespace
  • Add method parse_thin_event() on the StripeClient class to parse thin events.

stripe-openapi bot and others added 2 commits September 26, 2024 11:33
* Update generated code for v1261

* Update generated code for v1268

* Update generated code for v1268

---------

Co-authored-by: Stripe OpenAPI <105521251+stripe-openapi[bot]@users.noreply.github.com>
@ramya-stripe ramya-stripe changed the title Sdk release/next major Support for APIs in the new API version 2024-09-30 Sep 27, 2024
jar-stripe and others added 2 commits September 27, 2024 17:28
* changed the object_name for v2 events from "event" to "v2.core.event"
* improved debug output in stripe_webhook_handler example
lib/stripe/version.rb Outdated Show resolved Hide resolved
lib/stripe/resources/thin_event.rb Outdated Show resolved Hide resolved
lib/stripe/resources/quote.rb Show resolved Hide resolved
@ramya-stripe ramya-stripe changed the title Support for APIs in the new API version 2024-09-30 Support for APIs in the new API version 2024-09-30.acacia Oct 1, 2024
… internal/deprecated (#1461)

* Add meter events base as an option

* Deprecate raw_request

* Mark APIRequestor.request as deprecated as well

* Untype to pass sorbet for now

* ignore rubocop
@ramya-stripe ramya-stripe merged commit 39d8736 into master Oct 1, 2024
27 checks passed
@ramya-stripe ramya-stripe deleted the sdk-release/next-major branch October 1, 2024 16:36
prathmesh-stripe added a commit that referenced this pull request Oct 3, 2024
* Support for APIs in the new API version 2024-09-30.acacia (#1458)

* remove parseSnapshotEvent (#1463)

* Bump version to 13.0.0

* Fixed API Version

---------

Co-authored-by: Ramya Rao <100975018+ramya-stripe@users.noreply.github.com>
Co-authored-by: Ramya Rao <ramya@stripe.com>
Co-authored-by: Prathmesh Ranaut <prathmesh@stripe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants