-
-
Notifications
You must be signed in to change notification settings - Fork 113
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 Artist Lookup not sanitizing special characters in Artist's names #472
Comments
ah i see what the issue is, for example taking https://www.last.fm/music/+noredirect/Destiny%E2%80%99s+Child the correct artist name is with a normal apostrophe (0x27) https://www.last.fm/music/Destiny%27s+Child maybe gonic could try follow those redirects |
seems to work now with 2878b88 this will be availble in the docker to clear your local artist info cache for relevant entries try
and also delete your gonic cover cache dir |
Thank you @sentriz! |
gonic version:v0.16.2
if from docker, docker tag: latest
Hello,
I've noticed that the Last.fm Artist lookup will fail(rendering a default image) in the following cases:
Eagle‐Eye Cherry
:Sent URL(This isn't pasting correcting into Github) :
https://www.last.fm/music/Eagle%E2%80%90Eye+Cherry
Correct URL:
https://www.last.fm/music/Eagle-Eye+Cherry
Destiny’s Child
:Sent URL(This isn't pasting correcting into Github):
https://www.last.fm/music/+noredirect/Destiny%E2%80%99S+Child
Correct url:
https://www.last.fm/music/Destiny's+Child
orhttps://www.last.fm/music/Destiny%27s+Child
D.J. Jazzy Jeff & The Fresh Prince
:Sent URL:
https://www.last.fm/music/+noredirect/D.J.+Jazzy+Jeff+&+the+Fresh+Prince
Correct URL:
https://www.last.fm/music/DJ+Jazzy+Jeff+&+The+Fresh+Prince
Cédric Gervais
:Sent URL:
https://www.last.fm/music/C%C3%A9dric+Gervais
Correct URL:
https://www.last.fm/music/Cedric+Gervais
The text was updated successfully, but these errors were encountered: