search

YAMJ / yamj-v2

Yet Another Movie Jukebox (YAMJ) v2
GNU General Public License v3.0
28 stars 11 forks source link

RottenTomatoes over queries per second limit #2711

Closed Omertron closed 9 years ago

Omertron commented 9 years ago

Original issue 2714 created by Omertron on 2013-12-20T07:41:27.000Z:

What steps will reproduce the problem?

After enabling RottenTomatoes ratings I have several errors in logfile:

RottenTomatoesPlugin: Failed to get RottenTomatoes information: Account Over Queries Per Second Limit

RottenTomatoesPlugin: Failed to get RottenTomatoes information: com.fasterxml.jackson.core.JsonParseException: Illegal character ((CTRL-CHAR, code 31)): only regular white space (\r, \n, \t) is allowed between tokens at [Source: java.io.StringReader@44b7f7; line: 1, column: 2]

RottenTomatoesPlugin: Failed to get RottenTomatoes information: Gateway Timeout

What version/revision of YAMJ are you using? On what operating system? r3819 on Ubuntu Server 12.04

What skin are you using? Eversion with evstreamed

What device are you using YAMJ on? Networked Media Tank, PlayonHD, Other? A200

Omertron commented 9 years ago

Comment #1 originally posted by Omertron on 2013-12-20T09:37:32.000Z:

The issue with the "\r \n \t" has been fixed in an update to the RT API which is included in later revisions of YAMJ.

I will look into the rate limiting for the requests.

Omertron commented 9 years ago

Comment #2 originally posted by Omertron on 2013-12-24T20:36:23.000Z:

Fixed this so that the RT API will try 5 times when we hit the queries per second limit. More than that and it will still throw the error.

Omertron commented 9 years ago

Comment #3 originally posted by Omertron on 2013-12-29T18:02:11.000Z:

This issue was closed by revision r3862.