excon 0.27.3 → 0.27.4

data/Gemfile.lock

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

data/changelog.txt

@@ -1,3 +1,8 @@
+0.27.4 10/14/2013
+=================
+
+fix for https/port string values
+
 0.27.3 10/11/2013
 =================
 

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.27.3'
-  s.date              = '2013-10-11'
+  s.version           = '0.27.4'
+  s.date              = '2013-10-14'
   s.rubyforge_project = 'excon'
 
   ## Make sure your summary is short. The description may be as long

data/lib/excon.rb

@@ -131,6 +131,7 @@ module Excon
     def new(url, params = {})
       uri_parser = params[:uri_parser] || Excon.defaults[:uri_parser]
       uri = uri_parser.parse(url)
+      raise ArgumentError.new("Invalid URI: #{uri}") unless uri.scheme
       params = {
         :host       => uri.host,
         :path       => uri.path,

data/lib/excon/constants.rb

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

data/lib/excon/ssl_socket.rb

@@ -51,8 +51,8 @@ module Excon
       end
 
       if @data[:proxy]
-        request = 'CONNECT ' << @data[:host] << ':' << @data[:port] << Excon::HTTP_1_1
-        request << 'Host: ' << @data[:host] << ':' << @data[:port] << Excon::CR_NL
+        request = 'CONNECT ' << @data[:host] << @data[:connection].send(:port_string, @data) << Excon::HTTP_1_1
+        request << 'Host: ' << @data[:host] << @data[:connection].send(:port_string, @data) << Excon::CR_NL
 
         if @data[:proxy][:password] || @data[:proxy][:user]
           auth = ['' << @data[:proxy][:user].to_s << ':' << @data[:proxy][:password].to_s].pack('m').delete(Excon::CR_NL)

data/tests/errors_tests.rb

@@ -1,5 +1,14 @@
 Shindo.tests('HTTPStatusError request/response debugging') do
 
+  tests('new raises errors for bad URIs').returns(true) do
+    begin
+      Excon.new('foo')
+      false
+    rescue => err
+      err.to_s.include? 'foo'
+    end
+  end
+
   with_server('error') do
 
     tests('message does not include response or response info').returns(true) do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: excon
 version: !ruby/object:Gem::Version
-  version: 0.27.3
+  version: 0.27.4
   prerelease: 
 platform: ruby
 authors:
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-11 00:00:00.000000000 Z
+date: 2013-10-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -243,7 +243,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -196741291753928057
+      hash: -4082744813762939124
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: