link_thumbnailer 1.0.8 → 1.0.9

data/lib/link_thumbnailer/fetcher.rb

@@ -21,7 +21,9 @@ module LinkThumbnailer
         resp                        = http.request(self.url)
         case resp
         when Net::HTTPSuccess     then resp.body
-        when Net::HTTPRedirection then fetch(resp['location'], redirect_count + 1)
+        when Net::HTTPRedirection
+          location = resp['location'].start_with?('http') ? resp['location'] : "#{self.url.scheme}://#{self.url.host}#{resp['location']}"
+          fetch(location, redirect_count + 1)
         else resp.error!
         end
       end

data/lib/link_thumbnailer/version.rb

@@ -1,3 +1,3 @@
 module LinkThumbnailer
-  VERSION = "1.0.8"
+  VERSION = "1.0.9"
 end

data/link_thumbnailer.gemspec

@@ -16,7 +16,7 @@ Gem::Specification.new do |gem|
   gem.version       = LinkThumbnailer::VERSION
 
   gem.add_dependency(%q{nokogiri},            ['~> 1.5.5'])
-  gem.add_dependency(%q{hashie},              ['~> 1.2.0'])
+  gem.add_dependency(%q{hashie},              ['>= 1.2.0'])
   gem.add_dependency(%q{net-http-persistent}, ['~> 2.7'])
   gem.add_dependency(%q{rmagick},             ['~> 2.13.1'])
   gem.add_dependency(%q{json},                ['~> 1.7.6'])

data/spec/fetcher_spec.rb

@@ -40,20 +40,44 @@ describe LinkThumbnailer::Fetcher do
 
     context "when http redirection" do
 
-      let(:another_url) { 'http://bar.com' }
+      context "and relative uri" do
 
-      before do
-        stub_request(:get, url).to_return(status: 300, body: 'foo', headers: { 'Location' => another_url })
-        stub_request(:get, another_url).to_return(status: 200, body: 'bar', headers: {})
-      end
+        let(:another_url) { '/bar' }
+
+        before do
+          stub_request(:get, url).to_return(status: 300, body: 'foo', headers: { 'Location' => another_url })
+          stub_request(:get, url + another_url).to_return(status: 200, body: 'bar', headers: {})
+        end
+
+        it "returns body response" do
+          fetcher.fetch(url).should eq('bar')
+        end
+
+        it "sets fetcher url" do
+          fetcher.fetch(url)
+          fetcher.url.to_s.should eq(url + another_url)
+        end
 
-      it "returns body response" do
-        fetcher.fetch(url).should eq('bar')
       end
 
-      it "sets fetcher url" do
-        fetcher.fetch(url)
-        fetcher.url.to_s.should eq(another_url)
+      context "and absolute uri" do
+
+        let(:another_url) { 'http://bar.com' }
+
+        before do
+          stub_request(:get, url).to_return(status: 300, body: 'foo', headers: { 'Location' => another_url })
+          stub_request(:get, another_url).to_return(status: 200, body: 'bar', headers: {})
+        end
+
+        it "returns body response" do
+          fetcher.fetch(url).should eq('bar')
+        end
+
+        it "sets fetcher url" do
+          fetcher.fetch(url)
+          fetcher.url.to_s.should eq(another_url)
+        end
+
       end
 
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: link_thumbnailer
 version: !ruby/object:Gem::Version
-  version: 1.0.8
+  version: 1.0.9
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-27 00:00:00.000000000 Z
+date: 2013-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -32,7 +32,7 @@ dependencies:
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 1.2.0
   type: :runtime
@@ -40,7 +40,7 @@ dependencies:
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ~>
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: 1.2.0
 - !ruby/object:Gem::Dependency