Hello community, here is the log from the commit of package rubygem-excon for openSUSE:Factory checked in at 2015-02-27 11:00:03 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/rubygem-excon (Old) and /work/SRC/openSUSE:Factory/.rubygem-excon.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Package is "rubygem-excon" Changes: -------- --- /work/SRC/openSUSE:Factory/rubygem-excon/rubygem-excon.changes 2015-02-13 08:35:17.000000000 +0100 +++ /work/SRC/openSUSE:Factory/.rubygem-excon.new/rubygem-excon.changes 2015-02-27 11:00:38.000000000 +0100 @@ -1,0 +2,7 @@ +Thu Feb 26 05:29:52 UTC 2015 - coolo@suse.com + +- updated to version 0.44.3 + don't pass body when following GET redirects + fix error rescue case to properly reference error object + +------------------------------------------------------------------- Old: ---- excon-0.44.2.gem New: ---- excon-0.44.3.gem ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ rubygem-excon.spec ++++++ --- /var/tmp/diff_new_pack.XVZoKZ/_old 2015-02-27 11:00:39.000000000 +0100 +++ /var/tmp/diff_new_pack.XVZoKZ/_new 2015-02-27 11:00:39.000000000 +0100 @@ -24,7 +24,7 @@ # Name: rubygem-excon -Version: 0.44.2 +Version: 0.44.3 Release: 0 %define mod_name excon %define mod_full_name %{mod_name}-%{version} ++++++ excon-0.44.2.gem -> excon-0.44.3.gem ++++++ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/Gemfile.lock new/Gemfile.lock --- old/Gemfile.lock 2015-02-11 21:26:10.000000000 +0100 +++ new/Gemfile.lock 2015-02-25 01:46:25.000000000 +0100 @@ -1,7 +1,7 @@ PATH remote: . specs: - excon (0.44.2) + excon (0.44.3) GEM remote: http://rubygems.org/ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/changelog.txt new/changelog.txt --- old/changelog.txt 2015-02-11 21:26:10.000000000 +0100 +++ new/changelog.txt 2015-02-25 01:46:25.000000000 +0100 @@ -1,3 +1,9 @@ +0.44.3 02/24/2015 +================= + +don't pass body when following GET redirects +fix error rescue case to properly reference error object + 0.44.2 02/11/2015 ================= Files old/checksums.yaml.gz and new/checksums.yaml.gz differ diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/excon.gemspec new/excon.gemspec --- old/excon.gemspec 2015-02-11 21:26:10.000000000 +0100 +++ new/excon.gemspec 2015-02-25 01:46:25.000000000 +0100 @@ -13,8 +13,8 @@ ## 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.2' - s.date = '2015-02-11' + s.version = '0.44.3' + s.date = '2015-02-24' s.rubyforge_project = 'excon' ## Make sure your summary is short. The description may be as long @@ -145,6 +145,7 @@ tests/rackups/deflater.ru tests/rackups/proxy.ru tests/rackups/query_string.ru + tests/rackups/redirecting.ru tests/rackups/request_headers.ru tests/rackups/request_methods.ru tests/rackups/response_header.ru diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/excon/constants.rb new/lib/excon/constants.rb --- old/lib/excon/constants.rb 2015-02-11 21:26:10.000000000 +0100 +++ new/lib/excon/constants.rb 2015-02-25 01:46:25.000000000 +0100 @@ -1,6 +1,6 @@ module Excon - VERSION = '0.44.2' + VERSION = '0.44.3' CR_NL = "\r\n" diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/excon/middlewares/redirect_follower.rb new/lib/excon/middlewares/redirect_follower.rb --- old/lib/excon/middlewares/redirect_follower.rb 2015-02-11 21:26:10.000000000 +0100 +++ new/lib/excon/middlewares/redirect_follower.rb 2015-02-25 01:46:25.000000000 +0100 @@ -17,7 +17,11 @@ params = datum.dup params.delete(:stack) params.delete(:connection) - params[:method] = :get if [301, 302, 303].include? response[:status] + if [301, 302, 303].include?(response[:status]) + params[:method] = :get + params.delete(:body) + params[:headers].delete('Content-Length') + end params[:headers] = datum[:headers].dup params[:headers].delete('Authorization') params[:headers].delete('Proxy-Connection') diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/lib/excon/ssl_socket.rb new/lib/excon/ssl_socket.rb --- old/lib/excon/ssl_socket.rb 2015-02-11 21:26:10.000000000 +0100 +++ new/lib/excon/ssl_socket.rb 2015-02-25 01:46:25.000000000 +0100 @@ -52,7 +52,7 @@ begin ssl_context.cert_store.add_file(ca_file) rescue - Excon.display_warning("Excon unable to add file to cert store, ignoring: #{ca_file}\n[#{e.class}] #{e.message}") + Excon.display_warning("Excon unable to add file to cert store, ignoring: #{ca_file}\n[#{$!.class}] #{$!.message}") end end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/metadata new/metadata --- old/metadata 2015-02-11 21:26:10.000000000 +0100 +++ new/metadata 2015-02-25 01:46:25.000000000 +0100 @@ -1,7 +1,7 @@ --- !ruby/object:Gem::Specification name: excon version: !ruby/object:Gem::Version - version: 0.44.2 + version: 0.44.3 platform: ruby authors: - dpiddy (Dan Peterson) @@ -10,7 +10,7 @@ autorequire: bindir: bin cert_chain: [] -date: 2015-02-11 00:00:00.000000000 Z +date: 2015-02-24 00:00:00.000000000 Z dependencies: - !ruby/object:Gem::Dependency name: activesupport @@ -221,6 +221,7 @@ - tests/rackups/deflater.ru - tests/rackups/proxy.ru - tests/rackups/query_string.ru +- tests/rackups/redirecting.ru - tests/rackups/request_headers.ru - tests/rackups/request_methods.ru - tests/rackups/response_header.ru diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/tests/middlewares/redirect_follower_tests.rb new/tests/middlewares/redirect_follower_tests.rb --- old/tests/middlewares/redirect_follower_tests.rb 2015-02-11 21:26:10.000000000 +0100 +++ new/tests/middlewares/redirect_follower_tests.rb 2015-02-25 01:46:25.000000000 +0100 @@ -61,3 +61,20 @@ env_restore end + +Shindo.tests("Excon redirecting post request") do + env_init + + with_rackup('redirecting.ru') do + tests("request not have content-length and body").returns('ok') do + Excon.post( + 'http://127.0.0.1:9292', + :path => '/first', + :middlewares => Excon.defaults[:middlewares] + [Excon::Middleware::RedirectFollower], + :body => "a=Some_content" + ).body + end + end + + env_restore +end diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' '--exclude=.svnignore' old/tests/rackups/redirecting.ru new/tests/rackups/redirecting.ru --- old/tests/rackups/redirecting.ru 1970-01-01 01:00:00.000000000 +0100 +++ new/tests/rackups/redirecting.ru 2015-02-25 01:46:25.000000000 +0100 @@ -0,0 +1,23 @@ +require 'sinatra' +require 'json' +require File.join(File.dirname(__FILE__), 'webrick_patch') + +class App < Sinatra::Base + set :environment, :production + enable :dump_errors + + post('/first') do + redirect "/second" + end + + get('/second') do + post_body = request.body.read + if post_body == "" && request["CONTENT_LENGTH"].nil? + "ok" + else + JSON.pretty_generate(request.env) + end + end +end + +run App -- To unsubscribe, e-mail: opensuse-commit+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-commit+help@opensuse.org