excon 0.44.0 → 0.44.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 90759d86297bebbce2b0374cb730466df92321ec
-  data.tar.gz: 67e4b85deb2f2b52455b190fa4c3df120fb1c532
+  metadata.gz: 481bfd91390599d3d4e0dfc27838fddc748adbf3
+  data.tar.gz: 0a6a75c64591bcb78f4a763667913c0a89e4ae8c
 SHA512:
-  metadata.gz: bda8264156cb7f3983aaac5cd8327254cc0f06a1ec9f1dc701271798cf06bfe2c207e14fb55bdd9b6deb96ce15843c0b2c6b0f7a5fbb66ead49a21d199b9f976
-  data.tar.gz: 511d4b47dd504535f78a25c4a741fcaea3206ec466fe57c199c6ae8c4ee8d5111ca5a1f8a39cb3e80cfb6f25d7f226df74a2b748f4b2392e3ece823479e470e8
+  metadata.gz: e6584d40bdf128fb820816775fa9fffd5504facd85704e8dec3e41c98a209e5dc8ca918d1ad7fb4662ae0f83cb556cb4ab782742c208170d3d0e62fc68b66b6e
+  data.tar.gz: 1942bd6aca9b73b82eb8a9a9b00cefb6b438416eaf85b53d4985d407a6d5bedcbac55df0e655817ba1c5a34a83f3e62b538e6b48554f5185b1a482d16bd30fee

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    excon (0.44.0)
+    excon (0.44.1)
 
 GEM
   remote: http://rubygems.org/

data/changelog.txt

@@ -1,3 +1,8 @@
+0.44.1 02/01/2015
+=================
+
+fix issue with frozen strings in user/pass
+
 0.44.0 01/30/2015
 =================
 

data/excon.gemspec

@@ -13,8 +13,8 @@ Gem::Specification.new do |s|
   ## If your rubyforge_project name is different, then edit it and comment out
   ## the sub! line in the Rakefile
   s.name              = 'excon'
-  s.version           = '0.44.0'
-  s.date              = '2015-01-30'
+  s.version           = '0.44.1'
+  s.date              = '2015-02-02'
   s.rubyforge_project = 'excon'
 
   ## Make sure your summary is short. The description may be as long

data/lib/excon/constants.rb

@@ -1,6 +1,6 @@
 module Excon
 
-  VERSION = '0.44.0'
+  VERSION = '0.44.1'
 
   CR_NL = "\r\n"
 

data/lib/excon/utils.rb

@@ -56,7 +56,7 @@ module Excon
     # Splits a header value +str+ according to HTTP specification.
     def split_header_value(str)
       return [] if str.nil?
-      str = str.strip
+      str = str.dup.strip
       str.force_encoding('BINARY') if FORCE_ENC
       str.scan(%r'\G((?:"(?:\\.|[^"])+?"|[^",]+)+)
                     (?:,\s*|\Z)'xn).flatten
@@ -64,18 +64,21 @@ module Excon
 
     # Escapes HTTP reserved and unwise characters in +str+
     def escape_uri(str)
+      str = str.dup
       str.force_encoding('BINARY') if FORCE_ENC
       str.gsub(UNESCAPED) { "%%%02X" % $1[0].ord }
     end
 
     # Unescapes HTTP reserved and unwise characters in +str+
     def unescape_uri(str)
+      str = str.dup
       str.force_encoding('BINARY') if FORCE_ENC
       str.gsub(ESCAPED) { $1.hex.chr }
     end
 
     # Unescape form encoded values in +str+
     def unescape_form(str)
+      str = str.dup
       str.force_encoding('BINARY') if FORCE_ENC
       str.gsub!(/\+/, ' ')
       str.gsub(ESCAPED) { $1.hex.chr }

data/tests/basic_tests.rb

@@ -89,19 +89,27 @@ end
 Shindo.tests('Excon basics (Basic Auth Pass)') do
   with_rackup('basic_auth.ru') do
     basic_tests('http://test_user:test_password@127.0.0.1:9292')
+    tests('with frozen args').returns(200) do
+      user, pass, uri = ['test_user', 'test_password', 'http://127.0.0.1:9292'].map(&:freeze)
+      connection = Excon.new(uri, :user => user, :password => pass )
+      response = connection.request(:method => :get, :path => '/content-length/100')
+      response.status
+    end  
+  end
+end
 
-    tests('Excon basics (Basic Auth Fail)') do
-      cases = [
-        ['correct user, no password', 'http://test_user@127.0.0.1:9292'],
-        ['correct user, wrong password', 'http://test_user:fake_password@127.0.0.1:9292'],
-        ['wrong user, correct password', 'http://fake_user:test_password@127.0.0.1:9292'],
-      ]
-      cases.each do |desc,url|
-        tests("response.status for #{desc}").returns(401) do
-          connection = Excon.new(url)
-          response = connection.request(:method => :get, :path => '/content-length/100')
-          response.status
-        end
+Shindo.tests('Excon basics (Basic Auth Fail)') do
+  with_rackup('basic_auth.ru') do
+    cases = [
+      ['correct user, no password', 'http://test_user@127.0.0.1:9292'],
+      ['correct user, wrong password', 'http://test_user:fake_password@127.0.0.1:9292'],
+      ['wrong user, correct password', 'http://fake_user:test_password@127.0.0.1:9292']
+    ]
+    cases.each do |desc,url|
+      tests("response.status for #{desc}").returns(401) do
+        connection = Excon.new(url)
+        response = connection.request(:method => :get, :path => '/content-length/100')
+        response.status
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: excon
 version: !ruby/object:Gem::Version
-  version: 0.44.0
+  version: 0.44.1
 platform: ruby
 authors:
 - dpiddy (Dan Peterson)
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-30 00:00:00.000000000 Z
+date: 2015-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport