-
Notifications
You must be signed in to change notification settings - Fork 598
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
Last.FM returned an error: Rate Limit Exceeded #3207
Comments
Can confirm, last.fm is not fetching any image for me. |
looks like they changed it https://getsatisfaction.com/lastfm/topics/api-announcement-dac8oefw5vrxq |
We could change this to take an API key from the config file instead of the apps' hard-coded API key in https://github.com/rembo10/headphones/blob/master/headphones/lastfm.py I'll see if I can work on a rudimentary fix for this and have it pull from the ini file. |
I already tried to add my own api key, but the issue stays the same. But I'm no coder by any means... so not sure. |
Last.fm has completely removed the ability to pull artwork via the API. Check out the link in the second reply. |
Upon deeper inspection (yes - artwork is not pullable now) we are violating the number of requests per minute per IP. so an API key issue is not really the thing, it's the number of time we are asking for something per second. |
Not exactly. They removed it period. For everyone regardless of how many calls. The error message is missleading. "While we allow API users to have access to a lot of data through the API, it has always been against our API Terms of Use for third parties to use audio, audiovisual, images or artwork. In some instances, some data may have been indirectly accessible through a small number of API calls, and so as part of the recent API clean-up, we have corrected that anomaly." |
ChaosBlades, sorry I was not more clear - yes they removed a number of things from the API, including album artwork. That said, your assumption that because the artwork is missing from the JSON payload that we are getting a rate limit error is not quite correct. In the LastFM.py code we make a call to the API for as much data as we can get our greedy little paws on; artwork included (which fails). I suspect (but have not verified in the code) that upon failure to have the artwork, we are then retrying, multiple times for different albums, and thusly hitting the 5 calls per minute limit (or whatever it really is now since I can't find that posted anywhere) and now we get what we are perceiving as a failed call that is because the artwork is missing. As you can see in this log, there are a number of successful calls to the API before I get rate limited.
|
I've submitted a pull request that fixes this by switching artwork lookups to fanart.tv - #3214 |
If I change the API key then the only thing not working for me is Artist Image, everything else including Album image seems ok |
Any progress on this change? Nopoz edit seems to have failed testing |
Try the develop branch |
Nopz has fixed this in his pull request. I am currently running his fork
and everything works.
Thank You,
Frank Moss
Cloud Infrastructure Architect
* 314|292.9130
* frank@nine13tech.com
LinkedIn Profile <http://linkedin.com/in/nine13tech>
http://nine13tech.com
Projects of Note:
IBM SmartCloud (Now Enterprise)
<https://www-935.ibm.com/services/us/en/it-services/smartcloud-enterprise.html>
USTRANSCOM CDE Developer Cloud
<http://www.ustranscom.mil/foia/docs/AT21_Business_Process_Mgmt_%28BPM%29Spt-Dec11.pdf>
CenturyLink Private Cloud
<http://www.centurylink.com/business/enterprise/cloud/private-cloud-computing.html>
CenturyLink SAP S4/HANA on Cisco FLEXpod
<http://www.centurylink.com/business/enterprise/managed-hosting/sap-hana.html>
…On 8/16/19 6:40 PM, AdeHub wrote:
Try the develop branch
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#3207?email_source=notifications&email_token=ABFEWV2PAULGRJNQGMYXLNDQE43FRA5CNFSM4HL6ILUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4P525Q#issuecomment-522182006>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABFEWVYUI7JGCSKKM6M3G63QE43FRANCNFSM4HL6ILUA>.Web
Bug from
https://github.com/notifications/beacon/ABFEWVY5EN4KOVEO6ITHOLLQE43FRA5CNFSM4HL6ILUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4P525Q.gif
[ { ***@***.***": "http://schema.org", ***@***.***": "EmailMessage",
"potentialAction": { ***@***.***": "ViewAction", "target":
"#3207?email_source=notifications\u0026email_token=ABFEWV2PAULGRJNQGMYXLNDQE43FRA5CNFSM4HL6ILUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4P525Q#issuecomment-522182006","url":
"#3207?email_source=notifications\u0026email_token=ABFEWV2PAULGRJNQGMYXLNDQE43FRA5CNFSM4HL6ILUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4P525Q#issuecomment-522182006",
"name": "View Issue" }, "description": "View this Issue on GitHub",
"publisher": { ***@***.***": "Organization", "name": "GitHub", "url":
"https://github.com" } } ]
|
just tried his fork and its fine regarding metadata, but while switching rlses i get an error... |
I just added two artists and it didn't add either of their artwork. So I am pretty sure I didn't hit the rate limit. Doesn't look like this issue has been posted yet but everyone that uses Headphones should be getting it.
The text was updated successfully, but these errors were encountered: