url_fetcher 1.1.2 → 1.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: dc714df3af393f211fd3e12077866febbf042207
-  data.tar.gz: ba8d4c2950a2e578337f9b39e816c16638078bf1
+  metadata.gz: d402920ce67afd407a5fbf92187b5e76ec3ae8c9
+  data.tar.gz: a5493b15b2fa71c9f8be671c17098eebd4a02554
 SHA512:
-  metadata.gz: 3c6ab837a920bf0e01b637e07c2e8b08ae35db374649e1d0743afdd2e82981eb1c2b300c31e6cec6e9bc53197d9d49a853de366e40a1c19a23152b995c2bcdfe
-  data.tar.gz: 1edfda68b43d76138e98d37ea4715077f374c95f042f025706181f60e0b250ac3932b3936b9c7262e7281c16cfa9929c423e9e138375eb69db26f0575d665923
+  metadata.gz: aa95ce46a2425e428d3dca7f8f9204e5a0e2c5c773f34860485af92356ca6a3260287798322dde569b019715088145477a0fa81a0f20b30d68acfdf5d0338564
+  data.tar.gz: 73b1c388e02dcd0298e03512c69c49661793ed2d3992aa15191e1c9e772421552e05e01faf78d8c09ea0a3754b46096c8853bd231856e34649425f27a7325249

data/VERSION

@@ -1 +1 @@
-1.1.2
+1.1.3

data/lib/url_fetcher.rb

@@ -11,7 +11,7 @@ class UrlFetcher
   MEGABYTE = 1048576
   MAX_ATTEMPTS = 5.freeze
 
-  attr_reader :url
+  attr_reader :url, :resolved_url
 
   # Create a fetcher for the specified URL.
   #
@@ -74,6 +74,7 @@ class UrlFetcher
 
     previous_attempts << url
     uri = URI(url)
+    @resolved_url = uri.to_s
 
     http = Net::HTTP.new(uri.host, uri.port)
     http.read_timeout = options[:read_timeout] || 20 # This is seconds. Default is 60.

data/spec/url_fetcher_spec.rb

@@ -14,6 +14,7 @@ describe UrlFetcher do
     url_fetcher.close
     expect(url_fetcher).to be_closed
     expect(url_fetcher.body).to be_closed
+    expect(url_fetcher.resolved_url).to eq("http://example.com/test")
   end
 
   it "should perform a POST request" do
@@ -47,19 +48,19 @@ describe UrlFetcher do
     WebMock.stub_request(:get, "http://example.com/test1").to_return(:status => 301, :headers => {"Location" => "http://example.com/test2"})
     WebMock.stub_request(:get, "http://example.com/test2").to_return(:status => 200, :body => "Hello", :headers => {"Content-Length" => 5})
     url_fetcher = UrlFetcher.new("http://example.com/test1")
-
     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.read).to eql("Hello")
+    expect(url_fetcher.resolved_url).to eq("http://example.com/test2")
   end
 
   it "should not honor redirects if :follow_redirects == false" do
     WebMock.stub_request(:get, "http://example.com/test1").to_return(:status => 301, :headers => {"Location" => "http://example.com/test2"})
     url_fetcher = UrlFetcher.new("http://example.com/test1", :follow_redirects => false)
-
     expect(url_fetcher).not_to be_success
     expect(url_fetcher).to be_redirect
+    expect(url_fetcher.resolved_url).to eq("http://example.com/test1")
   end
 
   it "should call a block before each redirect with the new location" do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: url_fetcher
 version: !ruby/object:Gem::Version
-  version: 1.1.2
+  version: 1.1.3
 platform: ruby
 authors:
 - weheartit
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-14 00:00:00.000000000 Z
+date: 2015-10-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable