url_fetcher 1.1.6 → 1.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8523decc234223ef23d0ad21e091b8e02b10697d
-  data.tar.gz: 54d317200ebbd7921eef0def888b0932a23a2f2f
+SHA256:
+  metadata.gz: 40fafbb372a909feae773b1e7908aebd79b77988b7d1b161f3cf54ee9a90bd37
+  data.tar.gz: 0c6596f02ff0b78c123447c44f656cbd4c6a742fa6532a61c4544095ead36e37
 SHA512:
-  metadata.gz: 82a6284bcd4a7096f8828b950431adb7940d2c9c2ad1b8e932ce16bf912ba4dac5fde62065c93ba4045028b97f8b8415f6fccb5de8fdebd0a6ebec56c26e5674
-  data.tar.gz: ad131782b83ac319e3819139c278f8f705c4c9a8f36705a1418aa494d54669521a2c64bde94dab23fa2990750504d3b4844d4e6714e55d2dbef494ad5c804e44
+  metadata.gz: 6f40b40f4eca2c555495033453ea6ea063bf44e0886565a6e2f3d26f716b63f0d6484a45afa4d469d39c7bec400ce6ae5d6bae9378dbd1c3127e716f0c3ad051
+  data.tar.gz: b29f096c78f65550cc329beb5eb4944195aaeb6eb927d9e8519428eb813a1eac94c48ac01a011984790ee348095ba41856f783d653c6cb643b1a5d108dc6adc3

data/VERSION

@@ -1 +1 @@
-1.1.6
+1.1.7

data/lib/url_fetcher.rb

@@ -34,11 +34,11 @@ class UrlFetcher
   def body
     @response.body if success?
   end
-  
+
   def close
     body.close if body && !body.closed?
   end
-  
+
   def closed?
     body.closed? if body
   end
@@ -100,12 +100,12 @@ class UrlFetcher
         name = name.to_s
         values = Array(value)
         request[name] = values[0].to_s
-        values[1, values.length].each do |val|          
+        values[1, values.length].each do |val|
           request.add_field(name, val.to_s)
         end
       end
     end
-    
+
     response = http.request(request) do |resp|
       unless resp.is_a?(Net::HTTPSuccess) || resp.is_a?(Net::HTTPRedirection)
         resp.value # Raises an appropriate HTTP error
@@ -113,7 +113,8 @@ class UrlFetcher
       if resp.is_a?(Net::HTTPSuccess) && resp.class.body_permitted?
         content_length = resp["Content-Length"].to_i
         raise FileTooBig.new(content_length) if content_length > (options[:max_size] || 10 * MEGABYTE)
-        ext = File.extname(url)
+        ext = File.extname(url.sub(/\?.*/, ''))
+        ext = nil if ext && ext.size > 12
         tempfile = Tempfile.new(["url_fetcher", ext], :encoding => 'ascii-8bit')
         resp.read_body(tempfile)
         tempfile.rewind

data/spec/url_fetcher_spec.rb

@@ -32,7 +32,7 @@ describe UrlFetcher do
     expect(url_fetcher).to be_success
     expect(url_fetcher).not_to be_redirect
     expect(url_fetcher.header("content-length")).to eql("5")
-    expect(url_fetcher.body).to be_nil
+    expect(url_fetcher.body.read).to eq ""
   end
 
   it "should work with SSL" do
@@ -43,7 +43,7 @@ describe UrlFetcher do
     expect(url_fetcher.header("content-length")).to eql("5")
     expect(url_fetcher.body.read).to eql("Hello")
   end
-  
+
   it "should send custom headers" do
     WebMock.stub_request(:get, "https://example.com/test").with(:headers => {"Accept" => ["image/jpeg", "image/gif"]}).to_return(:status => 200, :body => "Hello", :headers => {"Content-Length" => 5})
     url_fetcher = UrlFetcher.new("https://example.com/test", :headers => {"Accept" => ["image/jpeg", "image/gif"]})

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: url_fetcher
 version: !ruby/object:Gem::Version
-  version: 1.1.6
+  version: 1.1.7
 platform: ruby
 authors:
 - weheartit
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-03 00:00:00.000000000 Z
+date: 2019-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -120,7 +120,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.7.6
 signing_key: 
 specification_version: 4
 summary: Fetch resources from the internetz with circular redirects support