onebox 1.5.38 → 1.5.39

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e1352380dded0936ef25582e1d72ea8f265ea275
-  data.tar.gz: 335b4a502bd143fd6b926d7607087699c096c989
+  metadata.gz: 7ea86816288cce59362b5168910f92f0d84e755c
+  data.tar.gz: 28a653fc5dd177f71f31658d2b5339dbb7a9df18
 SHA512:
-  metadata.gz: 61bc29488640968ffa24af64d7efc44869f29a2464f7cd6f5f82624e19bfa07cf9e0a13eb542fa230721cad22e39fe0156123fd52dde4591f401e6758553cab2
-  data.tar.gz: ae7c475fdec87a70d8788cf60a7bfa286df26c114ece3edc7993771202c19a9fcc8e239adca087f15feeca533fdf3b83e354207686b8ce01c186990c4321ce3f
+  metadata.gz: a8693498c5c04aec5349c03e86fe9d1155a229525c423086475369e0fd6d06ba7335a72546142bafa2c668758fbcd70ac9773a9585d7ad52ff9eb9290a01b6ac
+  data.tar.gz: 9c4db21c6bf36d9859fdcbd36bf3db9de3d625e5188a02853167ad75a96490aff1a95320b5bc35c1db5b18e3b4b602730ee12111f4171fc9b15734884f36beab

data/lib/onebox/engine/amazon_onebox.rb

@@ -11,7 +11,11 @@ module Onebox
       matches_regexp(/^https?:\/\/(?:www)\.amazon\.(?<tld>com|ca|de|it|es|fr|co\.jp|co\.uk|cn|in|com\.br)\//)
 
       def url
-        return "http://www.amazon.#{tld}/gp/aw/d/" + URI::encode(match[:id]) if match && match[:id]
+        if match && match[:id]
+          _url = URI("http://www.amazon.#{tld}/gp/aw/d/" + URI::encode(match[:id]))
+          _url.scheme = URI(@url).scheme
+          return _url.to_s
+        end
         @url
       end
 

data/lib/onebox/version.rb

@@ -1,3 +1,3 @@
 module Onebox
-  VERSION = "1.5.38"
+  VERSION = "1.5.39"
 end

data/spec/lib/onebox/engine/amazon_onebox_spec.rb

@@ -18,8 +18,7 @@ describe Onebox::Engine::AmazonOnebox do
     end
 
     it "matches canadian domains" do
-      resp = check_link("ca", "http://www.amazon.ca/Too-Much-Happiness-Alice-Munro-ebook/dp/B0031TZ98K/")
-
+      check_link("ca", "http://www.amazon.ca/Too-Much-Happiness-Alice-Munro-ebook/dp/B0031TZ98K/")
     end
 
     it "matches german domains" do
@@ -60,6 +59,16 @@ describe Onebox::Engine::AmazonOnebox do
 
   end
 
+  describe "#url" do
+    it "maintains the same http/https scheme as the requested URL" do
+      expect(described_class.new("https://www.amazon.fr/gp/product/B01BYD0TZM").url)
+        .to eq("https://www.amazon.fr/gp/aw/d/B01BYD0TZM")
+
+      expect(described_class.new("http://www.amazon.fr/gp/product/B01BYD0TZM").url)
+        .to eq("http://www.amazon.fr/gp/aw/d/B01BYD0TZM")
+    end
+  end
+
   describe "#to_html" do
     it "includes image" do
       expect(html).to include("http://ecx.images-amazon.com/images/I/51opYcR6kVL._SY400_.jpg")
@@ -76,6 +85,6 @@ describe Onebox::Engine::AmazonOnebox do
     it "includes title" do
       expect(html).to include("Seven Languages in Seven Weeks: A Pragmatic Guide to Learning Programming Languages (Pragmatic Programmers)")
     end
-
   end
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: onebox
 version: !ruby/object:Gem::Version
-  version: 1.5.38
+  version: 1.5.39
 platform: ruby
 authors:
 - Joanna Zeta
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-30 00:00:00.000000000 Z
+date: 2016-04-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json