stripe 5.27.0 → 5.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6f68f6b92029b2acf9c99eb5837af2aa39ff686877d189acc79f997c7f56c414
-  data.tar.gz: c650ded2ab2ee0062f5f5be7a6b0a78a3d2f91173576aea916aedd57fb1c6e33
+  metadata.gz: fc768f1967f8b5d8d925a101dffc820bd4dfa96155cecc9bd86926cb30f07cbe
+  data.tar.gz: 7dcd29fb83035f6f6af335bf3a4acde40ea4ce7e0e097413b9f76c696bb3e529
 SHA512:
-  metadata.gz: b6d8708699cc5911fdfb5a2b7f1fad2cb62f432d8421535d3ffe238f271becab4aa6e0f85c875ad1949a509f07f6a779a979e63f5a5ddcdbee456db9fdcd63f0
-  data.tar.gz: 846bf9d449649354d1f210fe461801ab009281c0583a70f3f7efe63efcf8e5de8e768e9d6332a50a4b5484acf185036b3c9bde937b2b4488689450549d2b40db
+  metadata.gz: e911a5c3138805c0aeb5abe270186e4d4219b8414e3336579cda3caa02ad32797d0d088cce8468156f4dff27a6ff55ac107419e7a0b01040eec36be8684073d7
+  data.tar.gz: 6fd38cd5ffde35f1065ce18104382f106eec79fb3494099e49723745ada158f7f63df1164653a6ada3fe0c62ae0ed764cbf9e40e878f4e3bb4dbd6c799cffbfa

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # 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
 

data/README.md

@@ -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

data/VERSION

@@ -1 +1 @@
-5.27.0
+5.28.0

data/lib/stripe.rb

@@ -71,6 +71,7 @@ module Stripe
     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=

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/stripe_configuration.rb

@@ -42,6 +42,7 @@ module Stripe
     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
 
@@ -72,6 +73,7 @@ module Stripe
 
       @open_timeout = 30
       @read_timeout = 80
+      @write_timeout = 30
 
       @api_base = "https://api.stripe.com"
       @connect_base = "https://connect.stripe.com"
@@ -109,6 +111,15 @@ module Stripe
       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

data/lib/stripe/version.rb

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

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/stripe_configuration_test.rb

@@ -16,6 +16,7 @@ module Stripe
         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
@@ -25,10 +26,12 @@ module Stripe
         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
 

data/test/stripe_test.rb

@@ -54,6 +54,19 @@ class StripeTest < Test::Unit::TestCase
       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

data/test/test_helper.rb

@@ -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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 5.27.0
+  version: 5.28.0
 platform: ruby
 authors:
 - Stripe