stripe-java v17

[ob-stripe]

Integration branch for the HTTP client refactoring work.

(⚠️ = breaking changes)

• Refactor form encoding (Refactor form encoding #857)
• ⚠️ Refactor request telemetry (Refactor request telemetry #870)
• Move HTTP request methods into new HttpClient class (Move HTTP request methods into new HttpClient class #872)
• Add StripeRequest object (Add StripeRequest object #878)
• Add HttpClient abstract class (Add HttpClient abstract class #879)
• ⚠️ Stop disabling the DNS cache (Stop disabling the DNS cache #880)
• Fix deprecation warnings (Fix deprecation warnings #895)
• Add HttpContent class (Add HttpContent class #896)
• Add Stopwatch class (Add Stopwatch class #897)
• Move all request properties in StripeRequest (Move all request properties in StripeRequest #898)
• ⚠️ Remove ApiResource.RequestType (Remove ApiResource.RequestType #899)
• Add support for automatic request retries (Add support for automatic request retries #900)
• Minor fixes (Minor fixes #902)
• StringUtils class & better API key validation (StringUtils class & better API key validation #928)
• ⚠️ Remove support for custom URLStreamHandler (Remove support for custom URLStreamHandler #927)
• ⚠️ Refactor HTTP headers handling (Refactor HTTP headers handling #931)
• ⚠️ Modernize StripeResponse (Modernize StripeResponse #932)
• Add maxNetworkRetries as a global and per-request setting (Add maxNetworkRetries as a global and per-request setting #934)
• ⚠️ Add StreamUtils class (Add StreamUtils class #935)
• ⚠️ Remove support for count and total_count in list objects (Remove support for count and total_count in list objects #936)
• ⚠️ Remove deprecated properties and parameters (Remove deprecated properties and parameters #938)
• Update README (Update README #939)

[rattrayalex-stripe]

Can we be sure to include docs on using the retry feature?

[ob-stripe]

Okay, I think is ~ready.

I would also like to refactor StripeResponseGetter at some point to turn it into a proper StripeClient that holds configuration values, but that's a bit of a larger refactor that will also require changes to codegen (since right now every resource class creates full URLs by directly relying on the global configuration, e.g. Stripe.getApiBase() + "/v1/customers").

There's no immediate benefit to doing that though, so I'll probably do that in the future when we introduce service classes. For now we can release this which gives us a cleaner/better foundation as well as support for automatic request retries.

[ob-stripe]

r? @brandur-stripe @richardm-stripe @remi-stripe
cc @stripe/api-libraries

All PRs have been individually reviewed, so this is really more of a last sweep to see if anything's amiss.

I've also drafted a migration guide to describe the new features and breaking changes: https://github.com/stripe/stripe-java/wiki/%5Bwip%5D-Migration-guide-for-v17.

If there are no objections, I will merge and release this later this week.

[brandur-stripe]

Excellent migration guide!! Looks perfect. One thing I noticed is that it's possible you meant the very last header ("Custom URLStreamHandler") to be an h4 instead of an h3, but I'm not even 100% sure about that.

[ob-stripe]

you meant the very last header ("Custom URLStreamHandler") to be an h4 instead of an h3

Good eye! Fixed, thanks :)

[ob-stripe]

Released as 17.0.0.