Skip to content

Conversation

@richardm-stripe
Copy link
Contributor

@richardm-stripe richardm-stripe commented Jun 2, 2023

Changelog

"⚠️" symbol highlights breaking changes.

⚠️ Changed

  • Change the default behavior to perform 1 reattempt on retryable request failures (previously the default was 0). ((next major) one network retry by default #1803)
  • Allow request-level options to disable retries. Previously, there was a bug where passing maxNetworkRetries: 0 to per-request options would still cause the request to fall back to using the maxNetworkRetries value specified on the client. The client value happened to be 0 by default before, but this bug might occur if you set the client option to a non-zero value, then passed maxNetworkRetries: 0 as a per-request option somewhere. (stripe-node v13 release #1808)

⚠️ Removed

  • Remove deprecated del method on Subcriptions. Please use the cancel method instead, which was introduced in v9.14.0:
// Before
stripe.subscriptions.del('sub_123);

// After
stripe.subscriptions.cancel('sub_123');

@richardm-stripe richardm-stripe force-pushed the sdk-release/next-major branch from ac9661b to c17a9b6 Compare June 2, 2023 19:17
@richardm-stripe richardm-stripe changed the title Release branch: stripe-node v13 stripe-node v13 release Aug 1, 2023
@anniel-stripe anniel-stripe merged commit 07a8d2a into master Aug 16, 2023
@anniel-stripe anniel-stripe deleted the sdk-release/next-major branch August 16, 2023 18:27
gurus00 pushed a commit to gurus00/stripe-node that referenced this pull request Feb 11, 2025
* Fix retry override behavior

* Add failing test

* (next major) one network retry by default

* adjust tests

* Cleanup

* Remove subscriptions.del test (stripe#1869)

Remove subscriptions del test

---------

Co-authored-by: anniel-stripe <[email protected]>
gurus00 pushed a commit to gurus00/stripe-node that referenced this pull request Feb 11, 2025
* Fix retry override behavior

* Add failing test

* (next major) one network retry by default

* adjust tests

* Cleanup

* Remove subscriptions.del test (stripe#1869)

Remove subscriptions del test

---------

Co-authored-by: anniel-stripe <[email protected]>
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.

3 participants