faraday 2.14.1 → 2.14.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8d5ef64533c80aaabadd55b5ea342b102fd652f5918039c7deff3b26c4ad4a67
-  data.tar.gz: 81ebbac6eb08b7f85ee7d8673e3274bbe110c0a865604a3df7294814ba469eb8
+  metadata.gz: 05f10cf15f3eb3f35b39edc67e14c374306d286f7b282ce86e131cab66ef9726
+  data.tar.gz: 17df5c4c9c054080f80038a1d8c6e1eaf88fd5d319c5ca1f04b10bc7bc0fe350
 SHA512:
-  metadata.gz: 50080c6a64a9c7b0775da8c3d63ece38a3e0946f36500665ee9be6fc547fc1a5313897ffe87fb1f08f802b6a69b8b11f46fb3a2a751ba8c00aab6910cebadbb4
-  data.tar.gz: '093b793d6b3f0ca6951a4c71ffd1f4b2603a1ea7d6efce304b165ae8124dd7cb64f7aa46b455a0ee5c9ba8e0f91ec931484dcdd7d2082dbae64448ee7aa6b2d7'
+  metadata.gz: 511bc21e7d0e4c4ecf866f84b434e6f38d6c4eada9d17148f25db9c6cf3194c4fe6fe600cc1b41d288e611e8dbfcbbbb5e2d711051a4340239741bb9dae54b73
+  data.tar.gz: e527f43b558124d58d4c4826dc46d0716a6689fec48dea4daf47a3943e1cae5c3612b2e490c956811d0d2ba501290548e85f2b0879bd924fd026566a2dc6056e

data/examples/client_test.rb

@@ -26,7 +26,7 @@ class Client
 end
 
 # Example API client test
-class ClientTest < Test::Unit::TestCase
+class ClientTest < Test::Unit::TestCase # rubocop:disable Style/OneClassPerFile
   def test_httpbingo_name
     stubs = Faraday::Adapter::Test::Stubs.new
     stubs.get('/api') do |env|

data/lib/faraday/adapter.rb

@@ -8,6 +8,12 @@ module Faraday
 
     CONTENT_LENGTH = 'Content-Length'
 
+    TIMEOUT_KEYS = {
+      read: :read_timeout,
+      open: :open_timeout,
+      write: :write_timeout
+    }.freeze
+
     # This module marks an Adapter as supporting parallel requests.
     module Parallelism
       attr_writer :supports_parallel
@@ -23,6 +29,7 @@ module Faraday
     end
 
     extend Parallelism
+
     self.supports_parallel = false
 
     def initialize(_app = nil, opts = {}, &block)
@@ -89,12 +96,6 @@ module Faraday
       end
       options[key] || options[:timeout]
     end
-
-    TIMEOUT_KEYS = {
-      read: :read_timeout,
-      open: :open_timeout,
-      write: :write_timeout
-    }.freeze
   end
 end
 

data/lib/faraday/connection.rb

@@ -481,9 +481,11 @@ module Faraday
       if url && !base.path.end_with?('/')
         base.path = "#{base.path}/" # ensure trailing slash
       end
+      url = url.to_s if url.respond_to?(:host)
       # Ensure relative url will be parsed correctly (such as `service:search` or `//evil.com`)
       url = "./#{url}" if url.respond_to?(:start_with?) &&
-                          (!url.start_with?('http://', 'https://', '/', './', '../') || url.start_with?('//'))
+                          (url.start_with?('//') ||
+                           !url.start_with?('http://', 'https://', '/', './', '../'))
       uri = url ? base + url : base
       if params
         uri.query = params.to_query(params_encoder || options.params_encoder)

data/lib/faraday/encoders/flat_params_encoder.rb

@@ -6,6 +6,7 @@ module Faraday
   module FlatParamsEncoder
     class << self
       extend Forwardable
+
       def_delegators :'Faraday::Utils', :escape, :unescape
     end
 

data/lib/faraday/encoders/nested_params_encoder.rb

@@ -170,6 +170,7 @@ module Faraday
       attr_accessor :sort_params, :array_indices
 
       extend Forwardable
+
       def_delegators :'Faraday::Utils', :escape, :unescape
     end
 

data/lib/faraday/error.rb

@@ -172,7 +172,7 @@ module Faraday
   # A unified client error for timeouts.
   class TimeoutError < ServerError
     def initialize(exc = 'timeout', response = nil)
-      super(exc, response)
+      super
     end
   end
 

data/lib/faraday/middleware.rb

@@ -65,7 +65,7 @@ module Faraday
       if app.respond_to?(:close)
         app.close
       else
-        warn "#{app} does not implement \#close!"
+        warn "#{app} does not implement #close!"
       end
     end
   end

data/lib/faraday/options/env.rb

@@ -78,7 +78,7 @@ module Faraday
     # @param value [Object] a value fitting Option.from(v).
     # @return [Env] from given value
     def self.from(value)
-      env = super(value)
+      env = super
       if value.respond_to?(:custom_members)
         env.custom_members.update(value.custom_members)
       end
@@ -90,7 +90,7 @@ module Faraday
       return self[current_body] if key == :body
 
       if in_member_set?(key)
-        super(key)
+        super
       else
         custom_members[key]
       end
@@ -105,7 +105,7 @@ module Faraday
       end
 
       if in_member_set?(key)
-        super(key, value)
+        super
       else
         custom_members[key] = value
       end

data/lib/faraday/options/proxy_options.rb

@@ -7,6 +7,7 @@ module Faraday
   #   class ProxyOptions < Options; end
   ProxyOptions = Options.new(:uri, :user, :password) do
     extend Forwardable
+
     def_delegators :uri, :scheme, :scheme=, :host, :host=, :port, :port=,
                    :path, :path=
 
@@ -29,7 +30,7 @@ module Faraday
         end
       end
 
-      super(value)
+      super
     end
 
     memoized(:user) { uri&.user && Utils.unescape(uri.user) }

data/lib/faraday/options/request_options.rb

@@ -12,7 +12,7 @@ module Faraday
       if key && key.to_sym == :proxy
         super(key, value ? ProxyOptions.from(value) : nil)
       else
-        super(key, value)
+        super
       end
     end
 

data/lib/faraday/options.rb

@@ -186,7 +186,7 @@ module Faraday
     def [](key)
       key = key.to_sym
       if (method = self.class.memoized_attributes[key])
-        super(key) || (self[key] = instance_eval(&method))
+        super || (self[key] = instance_eval(&method))
       else
         super
       end

data/lib/faraday/rack_builder.rb

@@ -15,6 +15,11 @@ module Faraday
     # Used to detect missing arguments
     NO_ARGUMENT = Object.new
 
+    LOCK_ERR = "can't modify middleware stack after making a request"
+    MISSING_ADAPTER_ERROR = "An attempt to run a request with a Faraday::Connection without adapter has been made.\n" \
+                            "Please set Faraday.default_adapter or provide one when initializing the connection.\n" \
+                            'For more info, check https://lostisland.github.io/faraday/usage/.'
+
     attr_accessor :handlers
 
     # Error raised when trying to modify the stack after calling `lock!`
@@ -211,11 +216,6 @@ module Faraday
 
     private
 
-    LOCK_ERR = "can't modify middleware stack after making a request"
-    MISSING_ADAPTER_ERROR = "An attempt to run a request with a Faraday::Connection without adapter has been made.\n" \
-                            "Please set Faraday.default_adapter or provide one when initializing the connection.\n" \
-                            'For more info, check https://lostisland.github.io/faraday/usage/.'
-
     def raise_if_locked
       raise StackLocked, LOCK_ERR if locked?
     end

data/lib/faraday/utils/headers.rb

@@ -51,7 +51,7 @@ module Faraday
 
       def [](key)
         key = KeyMap[key]
-        super(key) || super(@names[key.downcase])
+        super || super(@names[key.downcase])
       end
 
       def []=(key, val)
@@ -59,13 +59,13 @@ module Faraday
         key = (@names[key.downcase] ||= key)
         # join multiple values with a comma
         val = val.to_ary.join(', ') if val.respond_to?(:to_ary)
-        super(key, val)
+        super
       end
 
       def fetch(key, ...)
         key = KeyMap[key]
         key = @names.fetch(key.downcase, key)
-        super(key, ...)
+        super
       end
 
       def delete(key)
@@ -74,13 +74,13 @@ module Faraday
         return unless key
 
         @names.delete key.downcase
-        super(key)
+        super
       end
 
       def dig(key, *rest)
         key = KeyMap[key]
         key = @names.fetch(key.downcase, key)
-        super(key, *rest)
+        super
       end
 
       def include?(key)

data/lib/faraday/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Faraday
-  VERSION = '2.14.1'
+  VERSION = '2.14.2'
 end

data/spec/faraday/connection_spec.rb

@@ -318,6 +318,13 @@ RSpec.describe Faraday::Connection do
         expect(uri.host).to eq('httpbingo.org')
       end
 
+      it 'does not allow host override with URI("//evil.com/path")' do
+        conn.url_prefix = 'http://httpbingo.org/api'
+        uri = conn.build_exclusive_url(URI('//evil.com/path?token=1'))
+        expect(uri.host).to eq('httpbingo.org')
+        expect(uri.query).to eq('token=1')
+      end
+
       it 'does not allow host override with //evil.com:8080/path' do
         conn.url_prefix = 'http://httpbingo.org/api'
         uri = conn.build_exclusive_url('//evil.com:8080/path')

data/spec/faraday/middleware_spec.rb

@@ -52,14 +52,14 @@ RSpec.describe Faraday::Middleware do
   end
 
   describe '#close' do
-    context "with app that doesn't support \#close" do
+    context "with app that doesn't support #close" do
       it 'should issue warning' do
         is_expected.to receive(:warn)
         subject.close
       end
     end
 
-    context "with app that supports \#close" do
+    context 'with app that supports #close' do
       it 'should issue warning' do
         expect(app).to receive(:close)
         is_expected.to_not receive(:warn)

data/spec/faraday/request_spec.rb

@@ -31,6 +31,15 @@ RSpec.describe Faraday::Request do
     it { expect(subject.to_env(conn).url.to_s).to eq('http://httpbingo.org/api/foo.json?a=1') }
   end
 
+  context 'when setting the url on setup with a protocol-relative URI' do
+    let(:block) { proc { |req| req.url URI.parse('//evil.com/path?token=1') } }
+    let(:url) { subject.to_env(conn).url }
+
+    it { expect(url.host).to eq('httpbingo.org') }
+    it { expect(url.path).to eq('/api///evil.com/path') }
+    it { expect(url.query).to eq('token=1') }
+  end
+
   context 'when setting the url on setup with a string path and params' do
     let(:block) { proc { |req| req.url 'foo.json', 'a' => 1 } }
 

data/spec/faraday/response/json_spec.rb

@@ -106,7 +106,7 @@ RSpec.describe Faraday::Response::Json, type: :response do
     end
 
     it 'passes relevant options to JSON parse' do
-      expect(::JSON).to receive(:parse)
+      expect(JSON).to receive(:parse)
         .with(body, options[:parser_options])
         .and_return(result)
 

data/spec/faraday/response/logger_spec.rb

@@ -218,7 +218,7 @@ RSpec.describe Faraday::Response::Logger do
 
     it 'logs response body object' do
       conn.get '/rubbles', nil, accept: 'text/html'
-      expect(string_io.string).to match(%([\"Barney\", \"Betty\", \"Bam Bam\"]\n))
+      expect(string_io.string).to match(%(["Barney", "Betty", "Bam Bam"]\n))
     end
 
     it 'logs filter body' do

data/spec/faraday/utils_spec.rb

@@ -33,7 +33,7 @@ RSpec.describe Faraday::Utils do
     end
 
     it 'parses with URI' do
-      with_default_uri_parser(::URI) do
+      with_default_uri_parser(URI) do
         uri = normalize(url)
         expect(uri.host).to eq('example.com')
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: faraday
 version: !ruby/object:Gem::Version
-  version: 2.14.1
+  version: 2.14.2
 platform: ruby
 authors:
 - "@technoweenie"
@@ -144,7 +144,7 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://lostisland.github.io/faraday
-  changelog_uri: https://github.com/lostisland/faraday/releases/tag/v2.14.1
+  changelog_uri: https://github.com/lostisland/faraday/releases/tag/v2.14.2
   source_code_uri: https://github.com/lostisland/faraday
   bug_tracker_uri: https://github.com/lostisland/faraday/issues
   rubygems_mfa_required: 'true'