glyr icon indicating copy to clipboard operation
glyr copied to clipboard

[Patch included] Various lyrics providers broken

Open Galaxy102 opened this issue 3 years ago • 2 comments

Hello,

I was researching lyric tools when I came across your glyr library, which I found very interesting as base scraper. During testing, I found out that Magistrix didn't serve any lyrics even though a quick test revealed their availability. As it wasn't late enough to go to bed, I fiddled around a bit and performed the needed changes to extract the lyrics correctly (as of today). The patch is attached as I didn't want to fork the repo with the included API Keys. If you want to attribute me (I don't insist on it, but would be glad), please use the information from my profile.

Thanks for the library! If there are any other issues regarding Web APIs, I could imagine further supporting this project.

patch-magistrix-2021-05-21-galaxy102.txt

Galaxy102 avatar May 21 '21 00:05 Galaxy102

Here attached there is another patch for Vagalume where the API URL has changed.

patch-vagalume-2021-05-22-galaxy102.txt

Galaxy102 avatar May 21 '21 22:05 Galaxy102

I continued testing all remaining services. Lyricswiki, lyricsvip, lyricstime, lyrdb and lipwalk seem to be down. I wrote fixes patches for elyrics and lyricsreg attached below.

patch-lyricsreg-2021-05-22-galaxy102.txt patch-elyrics-2021-05-22-galaxy102.txt

Galaxy102 avatar May 21 '21 23:05 Galaxy102