search

tegon / traktflix

Trakt.tv + Netflix = :heart:
https://tegon.github.io/traktflix
MIT License
404 stars 45 forks source link

themoviedb.org API over request limit #102

Open R-K-H opened 5 years ago

R-K-H commented 5 years ago

When loading multiple pages the total number of requests is over the limit.

{
"status_code":25,
"status_message":"Your request count (54) is over the allowed limit of 40."
}

As per their documentation https://developers.themoviedb.org/3/getting-started/request-rate-limiting Our current limits are 40 requests every 10 seconds and are limited by IP address, not API key.

I would imagine adding in a fixed limit to prevent going over would be advised for performance and expected behavior.

rafaelgomesxyz commented 5 years ago

I was actually thinking about removing the TMDb API. It's only used for retrieving images, and I'm pretty sure I can retrieve images from the Netflix API.

R-K-H commented 5 years ago

I think that's sound, otherwise a check mark would do but I think the image is a nice touch to let people know what they have seen / are watching.

smoothlystable commented 4 years ago

Trakt doesn't serve images through their API anymore. But TMDb disabled their rate limit recently, so this shouldn't be an issue anymore.

rafaelgomesxyz commented 4 years ago

@smoothlystable Still, it would be wise to use a server to store the TMDb URLs in a database to reduce the number of requests. I'm going to be working on this soon.