pincers 0.7.7 → 0.7.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6da332a1a2cd06206a29878abce0860c7dd9714a
-  data.tar.gz: 77a17252be6f58e1e76adcd04158c2ce94c3cbd3
+  metadata.gz: 4526e8ccc6d3ead10746ee9d7e95e473d063c467
+  data.tar.gz: 70fc43f5a6664ccbef9b0235db56e22065472882
 SHA512:
-  metadata.gz: e34c26b569947c44307c280864268f6e7cbe855f35c9c265b188ef585a87d96e98b82973fc05230b679fbdbfb93c7a7bb9a95e688b55f7df4dd889ae0d5a34ec
-  data.tar.gz: 01c3d5999f23c5351a14aaeab86a516d11006f8f93ec963af24d6a5c39e655d9ea422c9808a949d10ab5a27acee83842feb0cd3e8dede0aff6e8c2aadbd7d7ca
+  metadata.gz: 2ed5d778279256c922d66d71f763849edfc99f7060d818365a7ea73ffae06c5e3f7adf724d77021a05babd234f611e54c3898b657128f97a2ba81012acfc40f8
+  data.tar.gz: a911e5c4a3cb633f16308e6c03b4b27501f5241a0d2a1261a04e2f8d1e7acbb9df7e716f7aa2401874695a2c10032e334350c98e613b8bd70c5a2f01f8236394

data/lib/pincers/http/request.rb

@@ -1,8 +1,7 @@
 module Pincers::Http
   class Request
 
-    attr_reader :method, :uri, :headers
-    attr_accessor :data
+    attr_accessor :method, :uri, :data, :headers
 
     def initialize(_method, _uri)
       @method = _method
@@ -51,6 +50,17 @@ module Pincers::Http
       end
     end
 
+    def clone_for_redirect(_location, _repeat = true)
+      clone = self.class.new(_repeat ? method : :get, _location)
+
+      if _repeat
+        clone.headers = clone.headers.clone
+        clone.data = data
+      end
+
+      clone
+    end
+
   private
 
     def default_encoding_for(_pairs)
@@ -59,4 +69,4 @@ module Pincers::Http
     end
 
   end
-end
+end

data/lib/pincers/http/session.rb

@@ -45,15 +45,15 @@ module Pincers::Http
     end
 
     def perform(_request)
-      perform_recursive _request, @redirect_limit, nil
+      perform_recursive _request, @redirect_limit
     end
 
   private
 
-    def perform_recursive(_request, _limit, _redirect)
+    def perform_recursive(_request, _limit)
       raise MaximumRedirectsError.new if _limit == 0
 
-      uri = _redirect || _request.uri
+      uri = _request.uri
       path = uri.request_uri.empty? ? '/' : uri.request_uri
 
       http_request = _request.native_type.new path
@@ -72,12 +72,17 @@ module Pincers::Http
       when Net::HTTPRedirection
         location = Utils.parse_uri(http_response['location'])
         location = URI.join(uri, location) if location.relative?
-        perform_recursive(_request, _limit - 1, location)
+        new_request = _request.clone_for_redirect(location, repeating_redirect?(http_response))
+        perform_recursive(new_request, _limit - 1)
       else
         handle_error_response Response.new(uri, http_response)
       end
     end
 
+    def repeating_redirect?(_req)
+      ["307", "308"].include?(_req.code)
+    end
+
     def connect(_uri)
       conn = Net::HTTP.new _uri.host, _uri.port || 80, @proxy_addr, @proxy_port
       conn.use_ssl = true if _uri.scheme == 'https'

data/lib/pincers/version.rb

@@ -1,3 +1,3 @@
 module Pincers
-  VERSION = "0.7.7"
+  VERSION = "0.7.8"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pincers
 version: !ruby/object:Gem::Version
-  version: 0.7.7
+  version: 0.7.8
 platform: ruby
 authors:
 - Ignacio Baixas
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-20 00:00:00.000000000 Z
+date: 2016-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -288,7 +288,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.6.1
 signing_key: 
 specification_version: 4
 summary: Web automation framework with multiple backend support