Amazon onebox issue

RaceProUK · 2015 年5 月 21 日 09: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 日 13:02

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

Test…

tgxworld · 2015 年6 月 15 日 16: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 日 16: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 日 21: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 日 15: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 日 18:09

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

tgxworld · 2015 年6 月 21 日 09:24

嗯，描述 meta 标签写得不错，但我不确定“在 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 日 14: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 日 19: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 日 01:04

The PR hasn’t been merged yet

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

codinghorror · 2015 年7 月 14 日 01:48

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

codinghorror · 2015 年7 月 25 日 03:35

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

techAPJ · 2016 年10 月 9 日 19: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	3903	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 日

相关话题