stripe 5.23.1 → 5.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 33f1cdd1289229d711bcba485944474bfc3b645aaf0402bf2d4236eff4a87a4e
-  data.tar.gz: d2e05e209525b15095fcdbbfacb9f3e05ce4d7eb902c06c19f468bbd5a192184
+  metadata.gz: fc768f1967f8b5d8d925a101dffc820bd4dfa96155cecc9bd86926cb30f07cbe
+  data.tar.gz: 7dcd29fb83035f6f6af335bf3a4acde40ea4ce7e0e097413b9f76c696bb3e529
 SHA512:
-  metadata.gz: 2dcb34a7af495d2a704d3b48c35b08b1680bb24ea541e46e90c247ca57689fbaf213f798ef018a9fa9ed17644e42e76b7666f8fac2dd104f495f460492c5fb99
-  data.tar.gz: 232b4bef2acfc1b452ee1314c67d6e688a01a5860e4be93ad49491b65506cc3e10763c3a1090e2aeaa5137a1438f42a8f7e4f7741f4e1909f894ddb5dcef3639
+  metadata.gz: e911a5c3138805c0aeb5abe270186e4d4219b8414e3336579cda3caa02ad32797d0d088cce8468156f4dff27a6ff55ac107419e7a0b01040eec36be8684073d7
+  data.tar.gz: 6fd38cd5ffde35f1065ce18104382f106eec79fb3494099e49723745ada158f7f63df1164653a6ada3fe0c62ae0ed764cbf9e40e878f4e3bb4dbd6c799cffbfa

data/.travis.yml

@@ -12,12 +12,10 @@ notifications:
   email:
     on_success: never
 
-sudo: false
-
 env:
   global:
     # If changing this number, please also change it in `test/test_helper.rb`.
-    - STRIPE_MOCK_VERSION=0.95.0
+    - STRIPE_MOCK_VERSION=0.101.0
 
 cache:
   directories:

data/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 5.28.0 - 2020-10-14
+* [#950](https://github.com/stripe/stripe-ruby/pull/950) Add configuration option for `write_timeout` for connections on Ruby 2.6+
+
+## 5.27.0 - 2020-10-14
+* [#951](https://github.com/stripe/stripe-ruby/pull/951) Add support for the Payout Reverse API
+
+## 5.26.0 - 2020-09-29
+* [#949](https://github.com/stripe/stripe-ruby/pull/949) Add support for the `SetupAttempt` resource and List API
+
+## 5.25.0 - 2020-09-02
+* [#944](https://github.com/stripe/stripe-ruby/pull/944) Add support for the Issuing Dispute Submit API
+
+## 5.24.0 - 2020-08-26
+* [#939](https://github.com/stripe/stripe-ruby/pull/939) Extract configurations into separate object
+* [#940](https://github.com/stripe/stripe-ruby/pull/940) Fix typo in documentation of `stripe_object.rb`
+
 ## 5.23.1 - 2020-08-05
 * [#936](https://github.com/stripe/stripe-ruby/pull/936) Rename API resource's `request` method
 

data/README.md

@@ -60,14 +60,14 @@ available in your [Stripe Dashboard][api-keys]. Set `Stripe.api_key` to its
 value:
 
 ```ruby
-require "stripe"
-Stripe.api_key = "sk_test_..."
+require 'stripe'
+Stripe.api_key = 'sk_test_...'
 
 # list customers
 Stripe::Customer.list()
 
 # retrieve single customer
-Stripe::Customer.retrieve("cus_123456789")
+Stripe::Customer.retrieve('cus_123456789')
 ```
 
 ### Per-request Configuration
@@ -82,38 +82,38 @@ require "stripe"
 Stripe::Customer.list(
   {},
   {
-    api_key: "sk_test_...",
-    stripe_account: "acct_...",
-    stripe_version: "2018-02-28",
+    api_key: 'sk_test_...',
+    stripe_account: 'acct_...',
+    stripe_version: '2018-02-28',
   }
 )
 
 Stripe::Customer.retrieve(
-  "cus_123456789",
+  'cus_123456789',
   {
-    api_key: "sk_test_...",
-    stripe_account: "acct_...",
-    stripe_version: "2018-02-28",
+    api_key: 'sk_test_...',
+    stripe_account: 'acct_...',
+    stripe_version: '2018-02-28',
   }
 )
 
 Stripe::Customer.retrieve(
   {
-    id: "cus_123456789",
+    id: 'cus_123456789',
     expand: %w(balance_transaction)
   },
   {
-    stripe_version: "2018-02-28",
-    api_key: "sk_test_...",
+    stripe_version: '2018-02-28',
+    api_key: 'sk_test_...',
   }
 )
 
 Stripe::Customer.capture(
-  "cus_123456789",
+  'cus_123456789',
   {},
   {
-    stripe_version: "2018-02-28",
-    api_key: "sk_test_...",
+    stripe_version: '2018-02-28',
+    api_key: 'sk_test_...',
   }
 )
 ```
@@ -136,7 +136,7 @@ method:
 ```ruby
 client = Stripe::StripeClient.new
 customer, resp = client.request do
-  Stripe::Customer.retrieve("cus_123456789",)
+  Stripe::Customer.retrieve('cus_123456789',)
 end
 puts resp.request_id
 ```
@@ -146,7 +146,7 @@ puts resp.request_id
 A proxy can be configured with `Stripe.proxy`:
 
 ```ruby
-Stripe.proxy = "https://user:pass@example.com:1234"
+Stripe.proxy = 'https://user:pass@example.com:1234'
 ```
 
 ### Configuring an API Version
@@ -155,7 +155,7 @@ By default, the library will use the API version pinned to the account making
 a request. This can be overridden with this global option:
 
 ```ruby
-Stripe.api_version = "2018-02-28"
+Stripe.api_version = '2018-02-28'
 ```
 
 See [versioning in the API reference][versioning] for more information.
@@ -166,7 +166,7 @@ By default, the library will use its own internal bundle of known CA
 certificates, but it's possible to configure your own:
 
 ```ruby
-Stripe.ca_bundle_path = "path/to/ca/bundle"
+Stripe.ca_bundle_path = 'path/to/ca/bundle'
 ```
 
 ### Configuring Automatic Retries
@@ -186,11 +186,12 @@ retries are safe.
 
 ### Configuring Timeouts
 
-Open and read timeouts are configurable:
+Open, read and write timeouts are configurable:
 
 ```ruby
 Stripe.open_timeout = 30 # in seconds
 Stripe.read_timeout = 80
+Stripe.write_timeout = 30 # only supported on Ruby 2.6+
 ```
 
 Please take care to set conservative read timeouts. Some API requests can take
@@ -258,7 +259,7 @@ For example:
 Stripe::Instrumentation.subscribe(:request_end) do |request_event|
   tags = {
     method: request_event.method,
-    resource: request_event.path.split("/")[2],
+    resource: request_event.path.split('/')[2],
     code: request_event.http_status,
     retries: request_event.num_retries
   }
@@ -272,7 +273,7 @@ If you're writing a plugin that uses the library, we'd appreciate it if you
 identified using `#set_app_info`:
 
 ```ruby
-Stripe.set_app_info("MyAwesomePlugin", version: "1.2.34", url: "https://myawesomeplugin.info");
+Stripe.set_app_info('MyAwesomePlugin', version: '1.2.34', url: 'https://myawesomeplugin.info')
 ```
 
 This information is passed along when the library makes calls to the Stripe

data/VERSION

@@ -1 +1 @@
-5.23.1
+5.28.0

data/lib/stripe.rb

@@ -12,6 +12,7 @@ require "securerandom"
 require "set"
 require "socket"
 require "uri"
+require "forwardable"
 
 # Version
 require "stripe/version"
@@ -38,6 +39,7 @@ require "stripe/error_object"
 require "stripe/api_resource"
 require "stripe/singleton_api_resource"
 require "stripe/webhook"
+require "stripe/stripe_configuration"
 
 # Named API resources
 require "stripe/resources"
@@ -48,47 +50,42 @@ require "stripe/oauth"
 module Stripe
   DEFAULT_CA_BUNDLE_PATH = __dir__ + "/data/ca-certificates.crt"
 
-  @app_info = nil
-
-  @api_base = "https://api.stripe.com"
-  @connect_base = "https://connect.stripe.com"
-  @uploads_base = "https://files.stripe.com"
-
-  @log_level = nil
-  @logger = nil
-
-  @proxy = nil
-
-  @max_network_retries = 0
-  @max_network_retry_delay = 2
-  @initial_network_retry_delay = 0.5
-
-  @ca_bundle_path = DEFAULT_CA_BUNDLE_PATH
-  @ca_store = nil
-  @verify_ssl_certs = true
+  # map to the same values as the standard library's logger
+  LEVEL_DEBUG = Logger::DEBUG
+  LEVEL_ERROR = Logger::ERROR
+  LEVEL_INFO = Logger::INFO
 
-  @open_timeout = 30
-  @read_timeout = 80
+  @app_info = nil
 
-  @enable_telemetry = true
+  @configuration = Stripe::StripeConfiguration.setup
 
   class << self
-    attr_accessor :api_key
-    attr_accessor :api_version
+    extend Forwardable
+
+    # User configurable options
+    def_delegators :@configuration, :api_key, :api_key=
+    def_delegators :@configuration, :api_version, :api_version=
+    def_delegators :@configuration, :stripe_account, :stripe_account=
+    def_delegators :@configuration, :api_base, :api_base=
+    def_delegators :@configuration, :uploads_base, :uploads_base=
+    def_delegators :@configuration, :connect_base, :connect_base=
+    def_delegators :@configuration, :open_timeout, :open_timeout=
+    def_delegators :@configuration, :read_timeout, :read_timeout=
+    def_delegators :@configuration, :write_timeout, :write_timeout=
+    def_delegators :@configuration, :proxy, :proxy=
+    def_delegators :@configuration, :verify_ssl_certs, :verify_ssl_certs=
+    def_delegators :@configuration, :ca_bundle_path, :ca_bundle_path=
+    def_delegators :@configuration, :log_level, :log_level=
+    def_delegators :@configuration, :logger, :logger=
+    def_delegators :@configuration, :max_network_retries, :max_network_retries=
+    def_delegators :@configuration, :enable_telemetry=, :enable_telemetry?
+
+    # Internal configurations
+    def_delegators :@configuration, :max_network_retry_delay
+    def_delegators :@configuration, :initial_network_retry_delay
+    def_delegators :@configuration, :ca_store
+
     attr_accessor :client_id
-    attr_accessor :stripe_account
-
-    # These all get manual attribute writers so that we can reset connections
-    # if they change.
-    attr_reader :api_base
-    attr_reader :connect_base
-    attr_reader :open_timeout
-    attr_reader :proxy
-    attr_reader :read_timeout
-    attr_reader :uploads_base
-    attr_reader :verify_ssl_certs
-
-    attr_reader :max_network_retry_delay, :initial_network_retry_delay
   end
 
   # Gets the application for a plugin that's identified some. See
@@ -101,126 +98,6 @@ module Stripe
     @app_info = info
   end
 
-  def self.api_base=(api_base)
-    @api_base = api_base
-    StripeClient.clear_all_connection_managers
-  end
-
-  # The location of a file containing a bundle of CA certificates. By default
-  # the library will use an included bundle that can successfully validate
-  # Stripe certificates.
-  def self.ca_bundle_path
-    @ca_bundle_path
-  end
-
-  def self.ca_bundle_path=(path)
-    @ca_bundle_path = path
-
-    # empty this field so a new store is initialized
-    @ca_store = nil
-
-    StripeClient.clear_all_connection_managers
-  end
-
-  # A certificate store initialized from the the bundle in #ca_bundle_path and
-  # which is used to validate TLS on every request.
-  #
-  # This was added to the give the gem "pseudo thread safety" in that it seems
-  # when initiating many parallel requests marshaling the certificate store is
-  # the most likely point of failure (see issue #382). Any program attempting
-  # to leverage this pseudo safety should make a call to this method (i.e.
-  # `Stripe.ca_store`) in their initialization code because it marshals lazily
-  # and is itself not thread safe.
-  def self.ca_store
-    @ca_store ||= begin
-      store = OpenSSL::X509::Store.new
-      store.add_file(ca_bundle_path)
-      store
-    end
-  end
-
-  def self.connect_base=(connect_base)
-    @connect_base = connect_base
-    StripeClient.clear_all_connection_managers
-  end
-
-  def self.enable_telemetry?
-    @enable_telemetry
-  end
-
-  def self.enable_telemetry=(val)
-    @enable_telemetry = val
-  end
-
-  # map to the same values as the standard library's logger
-  LEVEL_DEBUG = Logger::DEBUG
-  LEVEL_ERROR = Logger::ERROR
-  LEVEL_INFO = Logger::INFO
-
-  # When set prompts the library to log some extra information to $stdout and
-  # $stderr about what it's doing. For example, it'll produce information about
-  # requests, responses, and errors that are received. Valid log levels are
-  # `debug` and `info`, with `debug` being a little more verbose in places.
-  #
-  # Use of this configuration is only useful when `.logger` is _not_ set. When
-  # it is, the decision what levels to print is entirely deferred to the logger.
-  def self.log_level
-    @log_level
-  end
-
-  def self.log_level=(val)
-    # Backwards compatibility for values that we briefly allowed
-    if val == "debug"
-      val = LEVEL_DEBUG
-    elsif val == "info"
-      val = LEVEL_INFO
-    end
-
-    if !val.nil? && ![LEVEL_DEBUG, LEVEL_ERROR, LEVEL_INFO].include?(val)
-      raise ArgumentError,
-            "log_level should only be set to `nil`, `debug` or `info`"
-    end
-    @log_level = val
-  end
-
-  # Sets a logger to which logging output will be sent. The logger should
-  # support the same interface as the `Logger` class that's part of Ruby's
-  # standard library (hint, anything in `Rails.logger` will likely be
-  # suitable).
-  #
-  # If `.logger` is set, the value of `.log_level` is ignored. The decision on
-  # what levels to print is entirely deferred to the logger.
-  def self.logger
-    @logger
-  end
-
-  def self.logger=(val)
-    @logger = val
-  end
-
-  def self.max_network_retries
-    @max_network_retries
-  end
-
-  def self.max_network_retries=(val)
-    @max_network_retries = val.to_i
-  end
-
-  def self.open_timeout=(open_timeout)
-    @open_timeout = open_timeout
-    StripeClient.clear_all_connection_managers
-  end
-
-  def self.proxy=(proxy)
-    @proxy = proxy
-    StripeClient.clear_all_connection_managers
-  end
-
-  def self.read_timeout=(read_timeout)
-    @read_timeout = read_timeout
-    StripeClient.clear_all_connection_managers
-  end
-
   # Sets some basic information about the running application that's sent along
   # with API requests. Useful for plugin authors to identify their plugin when
   # communicating with Stripe.
@@ -234,16 +111,6 @@ module Stripe
       version: version,
     }
   end
-
-  def self.uploads_base=(uploads_base)
-    @uploads_base = uploads_base
-    StripeClient.clear_all_connection_managers
-  end
-
-  def self.verify_ssl_certs=(verify_ssl_certs)
-    @verify_ssl_certs = verify_ssl_certs
-    StripeClient.clear_all_connection_managers
-  end
 end
 
 Stripe.log_level = ENV["STRIPE_LOG"] unless ENV["STRIPE_LOG"].nil?

data/lib/stripe/connection_manager.rb

@@ -119,6 +119,9 @@ module Stripe
 
       connection.open_timeout = Stripe.open_timeout
       connection.read_timeout = Stripe.read_timeout
+      if connection.respond_to?(:write_timeout=)
+        connection.write_timeout = Stripe.write_timeout
+      end
 
       connection.use_ssl = uri.scheme == "https"
 

data/lib/stripe/object_types.rb

@@ -73,6 +73,7 @@ module Stripe
         Reversal::OBJECT_NAME => Reversal,
         Review::OBJECT_NAME => Review,
         SKU::OBJECT_NAME => SKU,
+        SetupAttempt::OBJECT_NAME => SetupAttempt,
         SetupIntent::OBJECT_NAME => SetupIntent,
         Sigma::ScheduledQueryRun::OBJECT_NAME => Sigma::ScheduledQueryRun,
         Source::OBJECT_NAME => Source,

data/lib/stripe/resources.rb

@@ -61,6 +61,7 @@ require "stripe/resources/reporting/report_run"
 require "stripe/resources/reporting/report_type"
 require "stripe/resources/reversal"
 require "stripe/resources/review"
+require "stripe/resources/setup_attempt"
 require "stripe/resources/setup_intent"
 require "stripe/resources/sigma/scheduled_query_run"
 require "stripe/resources/sku"

data/lib/stripe/resources/issuing/dispute.rb

@@ -9,6 +9,17 @@ module Stripe
       include Stripe::APIOperations::Save
 
       OBJECT_NAME = "issuing.dispute"
+
+      custom_method :submit, http_verb: :post
+
+      def submit(params = {}, opts = {})
+        request_stripe_object(
+          method: :post,
+          path: resource_url + "/submit",
+          params: params,
+          opts: opts
+        )
+      end
     end
   end
 end

data/lib/stripe/resources/payout.rb

@@ -10,6 +10,7 @@ module Stripe
     OBJECT_NAME = "payout"
 
     custom_method :cancel, http_verb: :post
+    custom_method :reverse, http_verb: :post
 
     def cancel(params = {}, opts = {})
       request_stripe_object(
@@ -19,5 +20,14 @@ module Stripe
         opts: opts
       )
     end
+
+    def reverse(params = {}, opts = {})
+      request_stripe_object(
+        method: :post,
+        path: resource_url + "/reverse",
+        params: params,
+        opts: opts
+      )
+    end
   end
 end

data/lib/stripe/resources/setup_attempt.rb

@@ -0,0 +1,10 @@
+# File generated from our OpenAPI spec
+# frozen_string_literal: true
+
+module Stripe
+  class SetupAttempt < APIResource
+    extend Stripe::APIOperations::List
+
+    OBJECT_NAME = "setup_attempt"
+  end
+end

data/lib/stripe/stripe_configuration.rb

@@ -0,0 +1,178 @@
+# frozen_string_literal: true
+
+module Stripe
+  # Configurable options:
+  #
+  # =ca_bundle_path=
+  # The location of a file containing a bundle of CA certificates. By default
+  # the library will use an included bundle that can successfully validate
+  # Stripe certificates.
+  #
+  # =log_level=
+  # When set prompts the library to log some extra information to $stdout and
+  # $stderr about what it's doing. For example, it'll produce information about
+  # requests, responses, and errors that are received. Valid log levels are
+  # `debug` and `info`, with `debug` being a little more verbose in places.
+  #
+  # Use of this configuration is only useful when `.logger` is _not_ set. When
+  # it is, the decision what levels to print is entirely deferred to the logger.
+  #
+  # =logger=
+  # The logger should support the same interface as the `Logger` class that's
+  # part of Ruby's standard library (hint, anything in `Rails.logger` will
+  # likely be suitable).
+  #
+  # If `.logger` is set, the value of `.log_level` is ignored. The decision on
+  # what levels to print is entirely deferred to the logger.
+  class StripeConfiguration
+    attr_accessor :api_key
+    attr_accessor :api_version
+    attr_accessor :client_id
+    attr_accessor :enable_telemetry
+    attr_accessor :logger
+    attr_accessor :stripe_account
+
+    attr_reader :api_base
+    attr_reader :uploads_base
+    attr_reader :connect_base
+    attr_reader :ca_bundle_path
+    attr_reader :log_level
+    attr_reader :initial_network_retry_delay
+    attr_reader :max_network_retries
+    attr_reader :max_network_retry_delay
+    attr_reader :open_timeout
+    attr_reader :read_timeout
+    attr_reader :write_timeout
+    attr_reader :proxy
+    attr_reader :verify_ssl_certs
+
+    def self.setup
+      new.tap do |instance|
+        yield(instance) if block_given?
+      end
+    end
+
+    # Create a new config based off an existing one. This is useful when the
+    # caller wants to override the global configuration
+    def reverse_duplicate_merge(hash)
+      dup.tap do |instance|
+        hash.each do |option, value|
+          instance.public_send("#{option}=", value)
+        end
+      end
+    end
+
+    def initialize
+      @ca_bundle_path = Stripe::DEFAULT_CA_BUNDLE_PATH
+      @enable_telemetry = true
+      @verify_ssl_certs = true
+
+      @max_network_retries = 0
+      @initial_network_retry_delay = 0.5
+      @max_network_retry_delay = 2
+
+      @open_timeout = 30
+      @read_timeout = 80
+      @write_timeout = 30
+
+      @api_base = "https://api.stripe.com"
+      @connect_base = "https://connect.stripe.com"
+      @uploads_base = "https://files.stripe.com"
+    end
+
+    def log_level=(val)
+      # Backwards compatibility for values that we briefly allowed
+      if val == "debug"
+        val = Stripe::LEVEL_DEBUG
+      elsif val == "info"
+        val = Stripe::LEVEL_INFO
+      end
+
+      levels = [Stripe::LEVEL_INFO, Stripe::LEVEL_DEBUG, Stripe::LEVEL_ERROR]
+
+      if !val.nil? && !levels.include?(val)
+        raise ArgumentError,
+              "log_level should only be set to `nil`, `debug` or `info`"
+      end
+      @log_level = val
+    end
+
+    def max_network_retries=(val)
+      @max_network_retries = val.to_i
+    end
+
+    def open_timeout=(open_timeout)
+      @open_timeout = open_timeout
+      StripeClient.clear_all_connection_managers
+    end
+
+    def read_timeout=(read_timeout)
+      @read_timeout = read_timeout
+      StripeClient.clear_all_connection_managers
+    end
+
+    def write_timeout=(write_timeout)
+      unless Net::HTTP.instance_methods.include?(:write_timeout=)
+        raise NotImplementedError
+      end
+
+      @write_timeout = write_timeout
+      StripeClient.clear_all_connection_managers
+    end
+
+    def proxy=(proxy)
+      @proxy = proxy
+      StripeClient.clear_all_connection_managers
+    end
+
+    def verify_ssl_certs=(verify_ssl_certs)
+      @verify_ssl_certs = verify_ssl_certs
+      StripeClient.clear_all_connection_managers
+    end
+
+    def uploads_base=(uploads_base)
+      @uploads_base = uploads_base
+      StripeClient.clear_all_connection_managers
+    end
+
+    def connect_base=(connect_base)
+      @connect_base = connect_base
+      StripeClient.clear_all_connection_managers
+    end
+
+    def api_base=(api_base)
+      @api_base = api_base
+      StripeClient.clear_all_connection_managers
+    end
+
+    def ca_bundle_path=(path)
+      @ca_bundle_path = path
+
+      # empty this field so a new store is initialized
+      @ca_store = nil
+
+      StripeClient.clear_all_connection_managers
+    end
+
+    # A certificate store initialized from the the bundle in #ca_bundle_path and
+    # which is used to validate TLS on every request.
+    #
+    # This was added to the give the gem "pseudo thread safety" in that it seems
+    # when initiating many parallel requests marshaling the certificate store is
+    # the most likely point of failure (see issue #382). Any program attempting
+    # to leverage this pseudo safety should make a call to this method (i.e.
+    # `Stripe.ca_store`) in their initialization code because it marshals lazily
+    # and is itself not thread safe.
+    def ca_store
+      @ca_store ||= begin
+                      store = OpenSSL::X509::Store.new
+                      store.add_file(ca_bundle_path)
+                      store
+                    end
+    end
+
+    def enable_telemetry?
+      enable_telemetry
+    end
+  end
+end

data/lib/stripe/stripe_object.rb

@@ -375,7 +375,7 @@ module Stripe
       begin
         super
       rescue NoMethodError => e
-        # If we notice the accessed name if our set of transient values we can
+        # If we notice the accessed name of our set of transient values we can
         # give the user a slightly more helpful error message. If not, just
         # raise right away.
         raise unless @transient_values.include?(name)

data/lib/stripe/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Stripe
-  VERSION = "5.23.1"
+  VERSION = "5.28.0"
 end

data/test/stripe/api_resource_test.rb

@@ -111,15 +111,15 @@ module Stripe
     end
 
     should "send expand when fetching through ListObject" do
-      stub_request(:get, "#{Stripe.api_base}/v1/customers/cus_123")
-        .to_return(body: JSON.generate(customer_fixture))
+      stub_request(:get, "#{Stripe.api_base}/v1/charges/ch_123")
+        .to_return(body: JSON.generate(charge_fixture))
 
-      stub_request(:get, "#{Stripe.api_base}/v1/customers/cus_123/sources/cc_test_card")
-        .with(query: { "expand" => ["customer"] })
-        .to_return(body: JSON.generate(customer_fixture))
+      stub_request(:get, "#{Stripe.api_base}/v1/charges/ch_123/refunds/re_123")
+        .with(query: { "expand" => ["balance_transaction"] })
+        .to_return(body: JSON.generate(charge_fixture))
 
-      customer = Stripe::Customer.retrieve("cus_123")
-      customer.sources.retrieve(id: "cc_test_card", expand: [:customer])
+      charge = Stripe::Charge.retrieve("ch_123")
+      charge.refunds.retrieve(id: "re_123", expand: [:balance_transaction])
     end
 
     context "when specifying per-object credentials" do

data/test/stripe/connection_manager_test.rb

@@ -39,6 +39,7 @@ module Stripe
 
         old_open_timeout = Stripe.open_timeout
         old_read_timeout = Stripe.read_timeout
+        old_write_timeout = Stripe.write_timeout
 
         begin
           # Make sure any global initialization here is undone in the `ensure`
@@ -47,6 +48,7 @@ module Stripe
 
           Stripe.open_timeout = 123
           Stripe.read_timeout = 456
+          Stripe.write_timeout = 789 if WRITE_TIMEOUT_SUPPORTED
 
           conn = @manager.connection_for("https://stripe.com")
 
@@ -63,6 +65,7 @@ module Stripe
           # Timeouts
           assert_equal 123, conn.open_timeout
           assert_equal 456, conn.read_timeout
+          assert_equal 789, conn.write_timeout if WRITE_TIMEOUT_SUPPORTED
 
           assert_equal true, conn.use_ssl?
           assert_equal OpenSSL::SSL::VERIFY_PEER, conn.verify_mode
@@ -72,6 +75,7 @@ module Stripe
 
           Stripe.open_timeout = old_open_timeout
           Stripe.read_timeout = old_read_timeout
+          Stripe.write_timeout = old_write_timeout if WRITE_TIMEOUT_SUPPORTED
         end
       end
 

data/test/stripe/customer_card_test.rb

@@ -5,6 +5,12 @@ require ::File.expand_path("../test_helper", __dir__)
 module Stripe
   class CustomerCardTest < Test::Unit::TestCase
     setup do
+      # Unfortunately, the OpenAPI spec has an issue where the sources list has the wrong
+      # url so we need to mock this call instead.
+      customer_json = { id: "cus_123", object: "customer", sources: { object: "list", data: [], has_more: true, url: "/v1/customers/cus_123/sources" } }
+      stub_request(:get, "#{Stripe.api_base}/v1/customers/cus_123")
+        .to_return(body: JSON.generate(customer_json))
+
       @customer = Stripe::Customer.retrieve("cus_123")
     end
 

data/test/stripe/customer_test.rb

@@ -57,7 +57,7 @@ module Stripe
       should "delete a discount" do
         customer = Stripe::Customer.retrieve("cus_123")
         customer = customer.delete_discount
-        assert_requested :delete, "#{Stripe.api_base}/v1/customers/#{customer.id}/discount"
+        assert_requested :delete, "#{Stripe.api_base}/v1/customers/cus_123/discount"
         assert customer.is_a?(Stripe::Customer)
       end
     end

data/test/stripe/issuing/dispute_test.rb

@@ -6,7 +6,7 @@ module Stripe
   module Issuing
     class DisputeTest < Test::Unit::TestCase
       should "be creatable" do
-        dispute = Stripe::Issuing::Dispute.create
+        dispute = Stripe::Issuing::Dispute.create(transaction: "ipi_123")
 
         assert_requested :post, "#{Stripe.api_base}/v1/issuing/disputes"
         assert dispute.is_a?(Stripe::Issuing::Dispute)
@@ -30,6 +30,25 @@ module Stripe
         assert_requested :post, "#{Stripe.api_base}/v1/issuing/disputes/ich_123"
         assert dispute.is_a?(Stripe::Issuing::Dispute)
       end
+
+      context "#submit" do
+        should "submit the dispute" do
+          dispute = Stripe::Issuing::Dispute.retrieve("idp_123")
+          dispute = dispute.submit
+          assert_requested :post,
+                           "#{Stripe.api_base}/v1/issuing/disputes/idp_123/submit"
+          assert dispute.is_a?(Stripe::Issuing::Dispute)
+        end
+      end
+
+      context ".submit" do
+        should "submit the dispute" do
+          dispute = Stripe::Issuing::Dispute.submit("idp_123")
+          assert_requested :post,
+                           "#{Stripe.api_base}/v1/issuing/disputes/idp_123/submit"
+          assert dispute.is_a?(Stripe::Issuing::Dispute)
+        end
+      end
     end
   end
 end

data/test/stripe/payout_test.rb

@@ -53,5 +53,20 @@ module Stripe
         assert payout.is_a?(Stripe::Payout)
       end
     end
+
+    context "#reverse" do
+      should "reverse a payout" do
+        payout = Stripe::Payout.retrieve("tr_123")
+        payout = payout.reverse
+        assert payout.is_a?(Stripe::Payout)
+      end
+    end
+
+    context ".reverse" do
+      should "reverse a payout" do
+        payout = Stripe::Payout.reverse("pm_123")
+        assert payout.is_a?(Stripe::Payout)
+      end
+    end
   end
 end

data/test/stripe/product_test.rb

@@ -19,8 +19,7 @@ module Stripe
 
     should "be creatable" do
       product = Stripe::Product.create(
-        name: "My Product",
-        type: "good"
+        name: "My Product"
       )
       assert_requested :post, "#{Stripe.api_base}/v1/products"
       assert product.is_a?(Stripe::Product)

data/test/stripe/setup_attempt_test.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../test_helper", __dir__)
+
+module Stripe
+  class SetupAttemptTest < Test::Unit::TestCase
+    should "be listable" do
+      setup_attempts = Stripe::SetupAttempt.list({
+        setup_intent: "seti_123",
+      })
+      assert_requested :get, "#{Stripe.api_base}/v1/setup_attempts?setup_intent=seti_123"
+      assert setup_attempts.data.is_a?(Array)
+      assert setup_attempts.data[0].is_a?(Stripe::SetupAttempt)
+    end
+  end
+end

data/test/stripe/stripe_configuration_test.rb

@@ -0,0 +1,131 @@
+# frozen_string_literal: true
+
+require ::File.expand_path("../test_helper", __dir__)
+
+module Stripe
+  class StripeConfigurationTest < Test::Unit::TestCase
+    context ".setup" do
+      should "initialize a new configuration with defaults" do
+        config = Stripe::StripeConfiguration.setup
+
+        assert_equal Stripe::DEFAULT_CA_BUNDLE_PATH, config.ca_bundle_path
+        assert_equal true, config.enable_telemetry
+        assert_equal true, config.verify_ssl_certs
+        assert_equal 2, config.max_network_retry_delay
+        assert_equal 0.5, config.initial_network_retry_delay
+        assert_equal 0, config.max_network_retries
+        assert_equal 30, config.open_timeout
+        assert_equal 80, config.read_timeout
+        assert_equal 30, config.write_timeout
+        assert_equal "https://api.stripe.com", config.api_base
+        assert_equal "https://connect.stripe.com", config.connect_base
+        assert_equal "https://files.stripe.com", config.uploads_base
+      end
+
+      should "allow for overrides when a block is passed" do
+        config = Stripe::StripeConfiguration.setup do |c|
+          c.open_timeout = 100
+          c.read_timeout = 100
+          c.write_timeout = 100 if WRITE_TIMEOUT_SUPPORTED
+        end
+
+        assert_equal 100, config.open_timeout
+        assert_equal 100, config.read_timeout
+        assert_equal 100, config.write_timeout if WRITE_TIMEOUT_SUPPORTED
+      end
+    end
+
+    context "#reverse_duplicate_merge" do
+      should "return a duplicate object with overrides" do
+        config = Stripe::StripeConfiguration.setup do |c|
+          c.open_timeout = 100
+        end
+
+        duped_config = config.reverse_duplicate_merge(read_timeout: 500)
+
+        assert_equal config.open_timeout, duped_config.open_timeout
+        assert_equal 500, duped_config.read_timeout
+      end
+    end
+
+    context "#max_network_retries=" do
+      should "coerce the option into an integer" do
+        config = Stripe::StripeConfiguration.setup
+
+        config.max_network_retries = "10"
+        assert_equal 10, config.max_network_retries
+      end
+    end
+
+    context "#log_level=" do
+      should "be backwards compatible with old values" do
+        config = Stripe::StripeConfiguration.setup
+
+        config.log_level = "debug"
+        assert_equal Stripe::LEVEL_DEBUG, config.log_level
+
+        config.log_level = "info"
+        assert_equal Stripe::LEVEL_INFO, config.log_level
+      end
+
+      should "raise an error if the value isn't valid" do
+        config = Stripe::StripeConfiguration.setup
+
+        assert_raises ArgumentError do
+          config.log_level = "Foo"
+        end
+      end
+    end
+
+    context "options that require all connection managers to be cleared" do
+      should "clear when setting allow ca_bundle_path" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.ca_bundle_path = "/path/to/ca/bundle"
+      end
+
+      should "clear when setting open timeout" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.open_timeout = 10
+      end
+
+      should "clear when setting read timeout" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.read_timeout = 10
+      end
+
+      should "clear when setting uploads_base" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.uploads_base = "https://other.stripe.com"
+      end
+
+      should "clearn when setting api_base to be configured" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.api_base = "https://other.stripe.com"
+      end
+
+      should "clear when setting connect_base" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.connect_base = "https://other.stripe.com"
+      end
+
+      should "clear when setting verify_ssl_certs" do
+        config = Stripe::StripeConfiguration.setup
+
+        StripeClient.expects(:clear_all_connection_managers)
+        config.verify_ssl_certs = false
+      end
+    end
+  end
+end

data/test/stripe_test.rb

@@ -23,28 +23,110 @@ class StripeTest < Test::Unit::TestCase
     end
   end
 
-  should "allow ca_bundle_path to be configured" do
-    begin
-      old = Stripe.ca_bundle_path
-      Stripe.ca_bundle_path = "path/to/ca/bundle"
-      assert_equal "path/to/ca/bundle", Stripe.ca_bundle_path
-    ensure
-      Stripe.ca_bundle_path = old
+  context "forwardable configurations" do
+    context "internal configurations" do
+      should "return the certificate store" do
+        assert Stripe.ca_store.is_a?(OpenSSL::X509::Store)
+      end
+
+      should "return the max_network_retry_delay" do
+        assert_equal 2, Stripe.max_network_retry_delay
+      end
+
+      should "return the initial_network_retry_delay" do
+        assert_equal 0.5, Stripe.initial_network_retry_delay
+      end
     end
-  end
 
-  should "allow max_network_retries to be configured" do
-    begin
-      old = Stripe.max_network_retries
-      Stripe.max_network_retries = 99
-      assert_equal 99, Stripe.max_network_retries
-    ensure
-      Stripe.max_network_retries = old
+    should "allow ca_bundle_path to be configured" do
+      Stripe::StripeClient.expects(:clear_all_connection_managers)
+      Stripe.ca_bundle_path = "/path/to/ca/bundle"
+      assert_equal "/path/to/ca/bundle", Stripe.ca_bundle_path
+    end
+
+    should "allow open timeout to be configured" do
+      Stripe.open_timeout = 10
+      assert_equal 10, Stripe.open_timeout
+    end
+
+    should "allow read timeout to be configured" do
+      Stripe.read_timeout = 10
+      assert_equal 10, Stripe.read_timeout
+    end
+
+    if WRITE_TIMEOUT_SUPPORTED
+      should "allow write timeout to be configured" do
+        Stripe.write_timeout = 10
+        assert_equal 10, Stripe.write_timeout
+      end
+    else
+      should "raise when write timeout to be configured is not supported" do
+        assert_raises NotImplementedError do
+          Stripe.write_timeout = 10
+        end
+      end
+    end
+
+    should "allow api_key to be configured" do
+      Stripe.api_key = "sk_local_test"
+      assert_equal "sk_local_test", Stripe.api_key
+    end
+
+    should "allow stripe_account to be configured" do
+      Stripe.stripe_account = "acct_1234"
+      assert_equal "acct_1234", Stripe.stripe_account
     end
-  end
 
-  should "have default open and read timeouts" do
-    assert_equal Stripe.open_timeout, 30
-    assert_equal Stripe.read_timeout, 80
+    should "allow enable_telemetry to be configured" do
+      begin
+        old = Stripe.enable_telemetry?
+
+        Stripe.enable_telemetry = false
+        assert_equal false, Stripe.enable_telemetry?
+      ensure
+        Stripe.enable_telemetry = old
+      end
+    end
+
+    should "allow log_level to be configured" do
+      Stripe.log_level = "debug"
+      assert_equal ::Logger::DEBUG, Stripe.log_level
+    end
+
+    should "allow logger to be configured" do
+      logger = Object.new
+      Stripe.logger = logger
+      assert_equal logger, Stripe.logger
+    end
+
+    should "allow proxy to be configured" do
+      Stripe.proxy = "http://proxy"
+      assert_equal "http://proxy", Stripe.proxy
+    end
+
+    should "allow uploads_base to be configured" do
+      Stripe.uploads_base = "https://other.stripe.com"
+      assert_equal "https://other.stripe.com", Stripe.uploads_base
+    end
+
+    should "allow api_base to be configured" do
+      Stripe.api_base = "https://other.stripe.com"
+      assert_equal "https://other.stripe.com", Stripe.api_base
+    end
+
+    should "allow connect_base to be configured" do
+      Stripe.connect_base = "https://other.stripe.com"
+      assert_equal "https://other.stripe.com", Stripe.connect_base
+    end
+
+    should "allow verify_ssl_certs to be configured" do
+      Stripe.verify_ssl_certs = false
+      assert_equal false, Stripe.verify_ssl_certs
+    end
+
+    should "allow client_id to be configured" do
+      Stripe.client_id = "client"
+      assert_equal "client", Stripe.client_id
+    end
   end
 end

data/test/test_helper.rb

@@ -16,7 +16,7 @@ require ::File.expand_path("test_data", __dir__)
 require ::File.expand_path("stripe_mock", __dir__)
 
 # If changing this number, please also change it in `.travis.yml`.
-MOCK_MINIMUM_VERSION = "0.95.0"
+MOCK_MINIMUM_VERSION = "0.101.0"
 MOCK_PORT = Stripe::StripeMock.start
 
 # Disable all real network connections except those that are outgoing to
@@ -56,6 +56,8 @@ module Test
       include Stripe::TestData
       include Mocha
 
+      WRITE_TIMEOUT_SUPPORTED = Net::HTTP.instance_methods.include?(:write_timeout=)
+
       setup do
         Stripe.api_key = "sk_test_123"
         Stripe.api_base = "http://localhost:#{MOCK_PORT}"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 5.23.1
+  version: 5.28.0
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-08-05 00:00:00.000000000 Z
+date: 2020-10-14 00:00:00.000000000 Z
 dependencies: []
 description: Stripe is the easiest way to accept payments online.  See https://stripe.com
   for details.
@@ -116,6 +116,7 @@ files:
 - lib/stripe/resources/reporting/report_type.rb
 - lib/stripe/resources/reversal.rb
 - lib/stripe/resources/review.rb
+- lib/stripe/resources/setup_attempt.rb
 - lib/stripe/resources/setup_intent.rb
 - lib/stripe/resources/sigma/scheduled_query_run.rb
 - lib/stripe/resources/sku.rb
@@ -138,6 +139,7 @@ files:
 - lib/stripe/resources/webhook_endpoint.rb
 - lib/stripe/singleton_api_resource.rb
 - lib/stripe/stripe_client.rb
+- lib/stripe/stripe_configuration.rb
 - lib/stripe/stripe_object.rb
 - lib/stripe/stripe_response.rb
 - lib/stripe/util.rb
@@ -206,11 +208,13 @@ files:
 - test/stripe/reporting/report_type_test.rb
 - test/stripe/reversal_test.rb
 - test/stripe/review_test.rb
+- test/stripe/setup_attempt_test.rb
 - test/stripe/setup_intent_test.rb
 - test/stripe/sigma/scheduled_query_run_test.rb
 - test/stripe/sku_test.rb
 - test/stripe/source_test.rb
 - test/stripe/stripe_client_test.rb
+- test/stripe/stripe_configuration_test.rb
 - test/stripe/stripe_object_test.rb
 - test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb
@@ -324,11 +328,13 @@ test_files:
 - test/stripe/reporting/report_type_test.rb
 - test/stripe/reversal_test.rb
 - test/stripe/review_test.rb
+- test/stripe/setup_attempt_test.rb
 - test/stripe/setup_intent_test.rb
 - test/stripe/sigma/scheduled_query_run_test.rb
 - test/stripe/sku_test.rb
 - test/stripe/source_test.rb
 - test/stripe/stripe_client_test.rb
+- test/stripe/stripe_configuration_test.rb
 - test/stripe/stripe_object_test.rb
 - test/stripe/stripe_response_test.rb
 - test/stripe/subscription_item_test.rb