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

fix(discord): Fix Album Art failing on Discord RPC #2666

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

hauntii
Copy link
Contributor

@hauntii hauntii commented Dec 3, 2024

Fixes the album art for uploaded tracks failing on Discord and appearing as a ? block or a ripped image paper.

Removes a .split that removes all query parameters from the album art links ( removing the query parameter ?sqp, necessary to authenticate the image url ) from the songInfo data handler. Unsure why this was added, if this breaks something somewhere else due to these query parameters being necessary to be removed, I would recommend this type of split is done there instead of making it completely inaccessible here.

Closes #2247 !

image

image

@JellyBrick JellyBrick added the bug Something isn't working label Dec 5, 2024
@JellyBrick
Copy link
Collaborator

I'm worried about adding sqp to the URL due to security.

@hauntii
Copy link
Contributor Author

hauntii commented Dec 11, 2024

I'm worried about adding sqp to the URL due to security.

Fair enough, not quite sure about the risks it entails other than access to the locker images.

Is it known what exactly sqp authenticates? If it's only these images I don't believe it's that bad considering you'd need the video link as well.

@JellyBrick
Copy link
Collaborator

@JellyBrick
Copy link
Collaborator

Could you provide thumbnail URL for uploaded tracks?

@JellyBrick JellyBrick merged commit 5070fd8 into th-ch:master Dec 20, 2024
3 of 6 checks passed
@JellyBrick JellyBrick changed the title fix: Fix Album Art failing on Discord RPC fix(discord): Fix Album Art failing on Discord RPC Dec 20, 2024
@JellyBrick JellyBrick added fix-available A fix to the issue is available in a new version and removed fix-available A fix to the issue is available in a new version labels Dec 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Discord Can't Fetch Album Art for Uploaded Audio Files (Rich Presence)
2 participants