Ubuntu Rest Scopes

Amazon scope hits backend too much for some department cases until it gets a 503

Bug #1403065 reported by Facundo Batista
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Rest Scopes
Fix Released
High
Facundo Batista

Bug Description

For example:

  https://dash.ubuntu.com/smartscopes/v2/amazon/search?q=&session_id=a0926175-c194-459a-ac1b-e05541f14a3d&query_id=0&platform=phone&department=Books%3A51546011&locale=en_US&filters=%7B%22sorting_primary_filter%22%3A%5B%22__featured__%22%5D%7D%0A

The problem is in the _get_department_info method, see the (incorrect) overwriting of 'node' attribute at the end of the method.

Related branches

lp:~facundo/ubuntu-rest-scopes/amazon-lp1403065
Roberto Alsina (community): Approve
Diff: 88 lines (+35/-11)
2 files modified
src/scopes/amazon.py (+17/-10)
src/scopes/tests/test_amazon.py (+18/-1)
Facundo Batista (facundo)
Changed in ubuntu-rest-scopes:
status: Triaged → In Progress
Revision history for this message
Facundo Batista (facundo) wrote :

Name overwriting wasn't the problem: just too frequent hits to backend.

Facundo Batista (facundo)
Changed in ubuntu-rest-scopes:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.