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

Fetchart searches having problems #1760

Closed
Nukien opened this issue Dec 11, 2015 · 2 comments
Closed

Fetchart searches having problems #1760

Nukien opened this issue Dec 11, 2015 · 2 comments
Labels
bug bugs that are confirmed and actionable

Comments

@Nukien
Copy link

Nukien commented Dec 11, 2015

I just noticed that cover art wasn't being found for some albums, so checked with -v

Google search uses old API that is no longer available

fetchart: trying source google for album Yazz - The Natural Life
fetchart: getting URL: https://ajax.googleapis.com/ajax/services/search/images?q=Yazz%2CThe+Natural+Life
&start=0&v=1.0
fetchart: error scraping art page
fetchart: Yazz - The Natural Life: no art found

So I tested some URLs directly ...

https://ajax.googleapis.com/ajax/services/search/images?q=Yazz%2CThe+Natural+Life&start=0&v=1.0
https://ajax.googleapis.com/ajax/services/search/images?q=The+Motels%2CAll+Four+One&start=0&v=1.0

and get an error page from google

{"responseData": null, "responseDetails": "This API is no longer available.", "responseStatus": 403}

iTunes fails with index out of range

fetchart: trying source itunes for album Yazz - The Natural Life
fetchart: iTunes search failed: list index out of range

I'm assuming that this is when it can't find that particular cover, so this one is probably not an issue. But it seems to happen a lot, and I thought iTunes had a good selection of cover art ...

@sampsyo sampsyo added the bug bugs that are confirmed and actionable label Dec 13, 2015
@sampsyo
Copy link
Member

sampsyo commented Dec 13, 2015

Thanks for looking into both of these! It looks like we'll need to remove the Google source (there's no obvious alternative without a complete overhaul) and look more closely at the iTunes source.

@sampsyo
Copy link
Member

sampsyo commented Dec 13, 2015

OK, I removed the Google backend. I also improved the logging for the iTunes backend so it will emit a more sensible message like "no results for query X" rather than just the exception's text. In a couple of experiments, the backend was still able to find a couple of images for me, so I think it's still (basically) working.

shamangeorge added a commit to shamangeorge/beets that referenced this issue Dec 16, 2015
* master: (94 commits)
  Clean up changelog for edit plugin
  scrub: Demote a log message to debug
  scrub: Restore tags & art in auto mode (beetbox#1657)
  scrub: Run on import in auto mode (beetbox#1657)
  Fix beetbox#1673: Escape regex terms in lyrics
  snake_case variable names
  Doc refinements for beetbox#1749
  Remove tests for Google fetchart backend (beetbox#1760)
  fetchart: Remove Google backend (fix beetbox#1760)
  fetchart: Better logging for iTunes (beetbox#1760)
  fetchart: Fix beetbox#1610: itunes install docs
  Fix test that depended on local time, 2
  Fix test that depended on local time
  Fix unused import leftover on test_library
  Add documentation for M:SS length
  Move raw_seconds_short to beets.util
  Add tests for library-specific field types
  Fix test that was expecting raw length format
  Fix pyflakes issues, variable name
  Add config option for human vs raw track duration
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug bugs that are confirmed and actionable
Projects
None yet
Development

No branches or pull requests

2 participants