Skip to content
This repository has been archived by the owner on Jan 22, 2022. It is now read-only.

Support album art mutation #38

Closed
simon-weber opened this issue May 10, 2012 · 9 comments
Closed

Support album art mutation #38

simon-weber opened this issue May 10, 2012 · 9 comments

Comments

@simon-weber
Copy link
Owner

Album art can be changed through the web interface, and should be a fairly straightforward feature to add. I don't have the time to implement it, though.

@simon-weber
Copy link
Owner Author

This is a pretty straightforward 2-part protocol.

First, upload an image:
url: https://play.google.com/music/services/imageupload
params:

  • u (0)
  • xt [token]
  • zx [12 alphanum chars? optional]

response: json {"imageUrl":"http://lh6.ggpht.com/[long id]s130-c-e100","imageDisplayUrl":"//lh6.googleusercontent.com/[long id]s130-c-e100"}

Then, use modifyentries to set albumArtUrl to the imageUrl provided in the last response.

@ralfharing
Copy link

I was having trouble updating the album art via the webpage and went looking to see if there was a way to do this programmatically (or see what was going wrong if there was really an error). Thanks for figuring this out! I saw in the readme that album art manipulation was "coming soon". Is there anything I can do to help?

@simon-weber
Copy link
Owner Author

I'm pretty set on the protocol right now, so all that's left to do is declare the calls and expose them with the api.

If you're feeling adventurous, feel free to take a stab at contributing: recent changes have made the protocol interface a lot nicer. The first step would be making a new WcCall (maybe UploadImage?) in https://github.com/simon-weber/Unofficial-Google-Music-API/blob/session-overhaul/gmusicapi/newprotocol/webclient.py. The docs for Call in https://github.com/simon-weber/Unofficial-Google-Music-API/blob/session-overhaul/gmusicapi/newprotocol/shared.py should make it pretty clear what's going on. Feedback is appreciated on this stuff, by the way; the goal is to make internals clearer so folks feel more comfortable contributing.

If you're not comfortable working on it: no problem. I'll get back in touch when I've got something you can test out. I don't have an ETA right now, but expect news within a week or so (I've got a few things this is waiting on).

@simon-weber
Copy link
Owner Author

This is implemented in 4bb85a9. It's not working just yet, since Music seems to be broken at the moment...I'll come back to this once I can change album art on the website successfully.

@simon-weber
Copy link
Owner Author

Wow, I can't believe this problem has gone on for so long. Thanks for finding these links.

@ralfharing
Copy link

fyi, it looks like the web site is working again

@simon-weber
Copy link
Owner Author

Great; thanks for letting me know.

@simon-weber
Copy link
Owner Author

Ok, I've finally got this working in 2c19848.

It's not heavily tested yet, but it seems to work.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants