Amazon onebox issue

RaceProUK · 2015 年 5 月 21 日午前 9:25

For some reason, the onebox doesn’t actually fill anything out
http://www.amazon.com/gp/product/B00H4SSB02

Doesn’t appear limited to just .com either:
http://www.amazon.co.uk/gp/product/B00H4SSB02

Edit: It seems the Amazon side is working OK:

accalia · 2015 年 5 月 21 日午前 11:12

Hmm. Testing full link (for a different product):

Stripping parameters:

Stripping ref:

huh. it seems to be something about the onebox information for that product.

tgxworld · 2015 年 6 月 15 日午後 1:02

http://www.amazon.com/gp/aw/d/193435659X

Test…

tgxworld · 2015 年 6 月 15 日午後 4:03

I think I fixed it for most of the products.

https://github.com/discourse/onebox/pull/304

http://www.amazon.com/gp/product/B00H4SSB02 seems to be a special case.

downey · 2015 年 6 月 15 日午後 4:31

Just a thought: It would be nice if this code also matched the smile.amazon.com domain (the rest of the URL is always the same) which allows non-profits to get a cut of the revenue.

riking · 2015 年 6 月 15 日午後 9:08

Submit a PR to expand the URL matching:

github.com/tgxworld/onebox

lib/onebox/engine/amazon_onebox.rb

507776c0d


      
          require 'json'
          
          module Onebox
            module Engine
              class AmazonOnebox
                include Engine
                include LayoutSupport
                include HTML
          
          
                matches_regexp(/^http:\/\/(?: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]
                  @url
                end
          
                def tld
                  @tld || @@matcher.match(@url)["tld"]
                end

tgxworld · 2015 年 6 月 17 日午後 3:29

Is there a reason we try to scrape using mobile pages from Amazon? The desktop pages contains a description meta tag that we can use to improve reliability and avoid breaking Amazon onebox everytime they change their HTML.

github.com/tgxworld/onebox

lib/onebox/engine/amazon_onebox.rb

fix_amazon_onebox


      
          module Engine
            class AmazonOnebox
              include Engine
              include LayoutSupport
              include HTML
          
          
              matches_regexp(/^http:\/\/(?: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]
                @url
              end
          
              def tld
                @tld || @@matcher.match(@url)["tld"]
              end
          
              def http_params
                {'User-Agent' => 'Mozilla/5.0 (iPhone; CPU iPhone OS 5_0_1 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A405 Safari/7534.48.3'}
              end

codinghorror · 2015 年 6 月 17 日午後 6:09

The idea was that mobile would have less cruft to sort through, if desktop is easier, go for it.

tgxworld · 2015 年 6 月 21 日午前 9:24

うーん、メタタグの説明はなかなか良いのですが、「Amazon.com で購入。対象商品なら送料無料」という部分が気になってしまいます。どう思われますか？

tgxworld · 2015 年 6 月 22 日午前 10:24

ふむ、meta タグの説明を使って試してみましたが、製品についてより多くの情報を提供するよりも、どちらかといえば広告のようだと感じました。

モバイルページに固執します。

cc/ @sam

codinghorror · 2015 年 6 月 26 日午前 10:10

Have we pulled in the updated onebox @eviltrout?

eviltrout · 2015 年 6 月 26 日午後 2:53

I just bumped it. I can’t say for sure if we were on the latest because our gemfile referenced a version that was pushed to Rubygems that wasn’t also set in our repo. I can confirm that now we are on the very latest onebox code.

neil · 2015 年 7 月 13 日午後 7:34

Test again:

Looks like this bug isn’t fixed, or we still aren’t deploying the latest version of onebox.

tgxworld · 2015 年 7 月 14 日午前 1:04

The PR hasn’t been merged yet

https://github.com/discourse/onebox/pull/304

codinghorror · 2015 年 7 月 14 日午前 1:48

Ok can we please merge that @neil? And merge in onebox version update too.

codinghorror · 2015 年 7 月 25 日午前 3:35

I think it is working now at last, thank you very much @tgxworld!

techAPJ · 2016 年 10 月 9 日午後 7:37

This is done now.

トピック		返信	表示
Amazon Onebox Broken? Bug	29	2682	2023 年 5 月 23 日
Amazon Onebox broken Bug onebox	9	502	2024 年 5 月 8 日
Amazon onebox fails on my site, but works here Bug	19	3904	2016 年 3 月 31 日
Onebox and Amazon not working (for me) Support	6	177	2025 年 1 月 2 日
Amazon links are failing to onebox Bug	2	447	2021 年 3 月 21 日

関連トピック