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

Extension stuck loading for songs without lyrics #966

Open
4 tasks done
AdrisAz opened this issue Nov 4, 2024 · 13 comments
Open
4 tasks done

Extension stuck loading for songs without lyrics #966

AdrisAz opened this issue Nov 4, 2024 · 13 comments
Assignees
Labels
Extension Bug Something isn't working

Comments

@AdrisAz
Copy link

AdrisAz commented Nov 4, 2024

How did you install Beautiful Lyrics?

Marketplace

What is the Bug/Issue?

The extension loads infinitely in both the sidebar and fullscreen mode when encountering a song without lyrics.

Screenshot 2024-11-03 221830
Screenshot 2024-11-03 221820
Screenshot 2024-11-03 222457

What Country are you from?

Canada

What Operating System are you using?

Windows

Spotify/Spicetify Information

Spotify Version: 1.2.49.439.gfae492c7
Spotify Runtime: cef_129.0.11+g57354b8+chromium-129.0.6668.90
Spicetify Version: 2.38.4
Spicetify Theme: marketplace
Spicetify Extensions: [beautiful-lyrics.js, hidePodcasts.js, goToSong.js, fullAlbumDate.js, playNextWrapper.js, imageOpener.js, CoverAmbience.js, oneko.js, copytoclipboard.js, made-for-you-shortcut.js, playbarClock.js, showQueueDuration.js, spotispy.js, shuffle+.js, AnonymizedRadios.js, cat-jam.js, beautiful-lyrics.mjs]
Spicetify Custom Apps: [marketplace]

Spotify HTML/CSS and Console Screenshots

Spotify.zip

Screenshot 2024-11-03 222403

Information Checklist

  • Spotify/Spicetify Information
  • Spotify HTML File
  • Spotify CSS File
  • Screenshots of Developer Tools Console Tab
@AdrisAz AdrisAz added the Extension Bug Something isn't working label Nov 4, 2024
@AdrisAz AdrisAz changed the title Summarize the Bug/Issue Here Extension stuck loading for songs without lyrics Nov 4, 2024
@zero-mindset
Copy link

I have this same issue (MacOS Sonoma 14.1) as well. This occurs on songs without lyrics, but also songs without synced lyrics

@AdrisAz
Copy link
Author

AdrisAz commented Nov 5, 2024

I think it's happening with non-synced lyrics for me too actually.

@AdrisAz AdrisAz closed this as completed Nov 5, 2024
@AdrisAz AdrisAz reopened this Nov 5, 2024
@SpikeNew7774
Copy link

It must've happened due to the server returning a 500 Errror status Code. Let me explain.
I've checked out the extension code, and found out that when the "https://beautiful-lyrics.socalifornian.live" server returns a different status code than 200, the extension doesn't know what to do and then it never hides the loading status. The only way to report it to the extension, that there are no lyrics, the server would have to return an empty body with the status code of 200.

photo

@HeyitsRadinn
Copy link

+1 same issue for me, multiple computers and networks

@SpikeNew7774
Copy link

I mean you could edit the actual built file for beautiful lyrics and do all that, but If you are a dev, you could just make your own beautiful lyrics. (im in the middle of doing that)

@zero-mindset
Copy link

zero-mindset commented Nov 5, 2024 via email

@SpikeNew7774
Copy link

@zero-mindset Do you mean that in the Beautiful-Lyrics extension or in the one that I'm making (I'm thinking of publishing it later)

@zero-mindset
Copy link

zero-mindset commented Nov 5, 2024 via email

@SpikeNew7774
Copy link

I could have a look into that, but if you would want it, it would need to be externally installed and not through the marketplace. I'm probably going to edit the actual output file and not the source code in this github repo -> (PS: Its tooooooooooo complicated for me).

@Lebbbbb
Copy link

Lebbbbb commented Nov 11, 2024

Anyone find a fix for this?

@SpikeNew7774
Copy link

SpikeNew7774 commented Nov 11, 2024

Kind of. Its a problem on the server, which could be fixed by adding a check if the response status isnt 200 to just hide the lyrics in the actual client code. Also I found out that if the lyrics are static on apple music, it will return a 500 error with the content about a 429 error. Response Body: "Stage: AM_S, Error: 429 | Too Many Requests"

@httpeace33
Copy link

httpeace33 commented Nov 13, 2024

I've got same issue for few days for almost all lyrics tho(

@SpikeNew7774
Copy link

That means their API must be failing. I'm currently making my own Apple Musix => JSON API that's compatible with the beautiful-lyrics format (I'm using it in my own Lyrics extension)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Extension Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants