Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[amazonpricebridge] Fixes for subscription items #2205

Merged
merged 2 commits into from Jul 12, 2021

Conversation

captn3m0
Copy link
Contributor

@captn3m0 captn3m0 commented Jul 8, 2021

This adds a few more CSS selectors as fallbacks for subscription items.
Note that this is tough to test for, since Amazon doesn't return pricing
information if the request is made from a different country.

Fixes #2203

This adds a few more CSS selectors as fallbacks for subscription items.
Note that this is tough to test for, since Amazon doesn't return pricing
information if the request is made from a different country.
@em92
Copy link
Contributor

em92 commented Jul 8, 2021

@mada199122, could you test this PR?
Temporarly deployed it here https://feed.eugenemolotov.ru/pr2205/

@mada199122
Copy link

tried now...
it doesn't work...

"Price: not available at the moment"

Screenshot 2021-07-08 at 21 01 47

@em92
Copy link
Contributor

em92 commented Jul 8, 2021

"Price: not available at the moment"

I see the same in original page.

Capture 4

@mada199122
Copy link

What?
Amazon is kidding us...
I see this instead...

Screenshot 2021-07-08 at 21 14 07

Is there a way to try your fix on my instance?

@em92
Copy link
Contributor

em92 commented Jul 8, 2021

My screenshot is made on my home laptop and I am using Russian ISP. Also my public instance is on Netherlands.

I guess you need to deploy RSS-Bridge instance on Italian location and apply this PR (excluding patch that I mentioned here #2205 (comment)) there to check it.

@mada199122
Copy link

sorry could you explain just a little bit how to apply this PR...
I'm not super skilled...
I've just pulled the latest version released some minutes ago from docker hub..

that version stil not work...

Screenshot 2021-07-08 at 21 38 30

@dhuschde
Copy link
Contributor

dhuschde commented Jul 8, 2021

sorry could you explain just a little bit how to apply this PR...
I'm not super skilled...
I've just pulled the latest version released some minutes ago from docker hub..

that version stil not work...

Screenshot 2021-07-08 at 21 38 30

If you pulled it only a few minutes ago, the Feed could still be in the Bridges cache

@mada199122
Copy link

uhm...
tried an other item but it doesn't work too:(

the item is this one...

https://www.amazon.it/Braun-Cartucce-Ricarica-Confezione-Ricambi/dp/B075RGQGZM

on your instance seams to work instead...

Screenshot 2021-07-08 at 21 55 24

@captn3m0
Copy link
Contributor Author

captn3m0 commented Jul 8, 2021

In my tests, I'd noticed that this is hard to test because Amazon hides the price information if you try accessing it from a different country.

Co-authored-by: Max <msal.coding@gmail.com>

Pulled in from RSS-Bridge#2204
@mada199122
Copy link

Can be helpful if I download an HTML of the Italian store from Italy and I attach here?

@captn3m0
Copy link
Contributor Author

captn3m0 commented Jul 8, 2021

Yes please. I've been testing against the HTML that I get after I set a PIN on the Amazon UI. (which does seem to work:

image
)

@mada199122
Copy link

really thanks
I've attached the HTML and the web archive saved with Safari

Amazon_italy_page.gz

@captn3m0
Copy link
Contributor Author

captn3m0 commented Jul 8, 2021

Tested this locally, and it works against the HTML you provided:

image

Let me get you a docker image so you can test this locally.

@mada199122
Copy link

ok... thanks

@captn3m0
Copy link
Contributor Author

captn3m0 commented Jul 8, 2021

It's already built: docker pull captn3m0/rss-bridge:issue-2203

@captn3m0
Copy link
Contributor Author

@mada199122 Any chance you could test this?

@mada199122
Copy link

really really sorry I thought to have answered you...

Yes it works perfectly that build!!!!

@captn3m0 captn3m0 requested a review from em92 July 12, 2021 13:41
@em92 em92 merged commit 9fa7821 into RSS-Bridge:master Jul 12, 2021
@em92
Copy link
Contributor

em92 commented Jul 12, 2021

gj @captn3m0 !

@mada199122
Copy link

with the last release now I have this response randomly
Screenshot 2021-07-14 at 14 05 01

@captn3m0 captn3m0 deleted the issue-2203 branch July 14, 2021 14:54
@captn3m0
Copy link
Contributor Author

I checked the HTML at https://www.amazon.it/dp/B07XK7QXS9 and the code should work fine with it.

This might be happening because Amazon is returning an error page? Amazon was facing issues recently with product pages so that might be it?

I see it picked up the product title correctly. @em92 Any suggestions on how to debug this?

@mada199122
Copy link

I think is something related to Amazon side... when RSS-Bridge does lot of query Amazon server return that...

I saw something similar months ago when I was trying to use Huginn...

Which user agent do you declare in the query?
In Huginn I had to change the User agent to a Chrome User agent...

@em92
Copy link
Contributor

em92 commented Jul 15, 2021

I see it picked up the product title correctly. @em92 Any suggestions on how to debug this?

I would modifiy bridge on your instance to save htmls and to save parsed prices. Using file_put_contents.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Amazon price tracker doesn't show price for some items in Italian store
4 participants