-
-
Notifications
You must be signed in to change notification settings - Fork 236
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
Switch from ytdl-core to play-dl #1031
Conversation
📦 A new release has been made for this pull request.To play around with this PR, pull Images are available for x86_64 and ARM64.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thank you!
I tested this out and it seems like tracks aren't being cached anymore.
I'd also like to replace ytsr/spotify-web-api-node with play-dl, but that can be a separate PR if you want.
That's strange. For me, caching works. I also checked it with the IDE debugger. Maybe you could send me the track you tested.
I will look into that. Edit: Btw you should update muse's spotify-uri dependency because the spotify urls are now localized and will always return 'No video found'. The new version will ignore the localization. |
Hello friend, I tried with the following docker, but no sound reaches Discord :/ version: '3.4'
services:
muse:
image: codetheweb/muse:pr-1031
restart: always
volumes:
- ./muse:/data
env_file: .env Logs
That URL (rr3---blabla...) returns an HTTP 403 Forbidden error code. |
Hi, sadly play-dl broke two weeks ago because YouTube changed something. We have to wait for a fix. |
@MarcoCoreDuo |
Hi, I think play-dl is still broken (no updates) and I can't really debug because of the "Sign in to confirm you’re not a bot" error. I will close this for now. |
Closes #1018
Should also resolve ytdl-core Issues #979 #945 #936
Play-dl doesn't directly offer ts formats from livestreams like ytdl-core does, so I had to fetch and parse them myself from the m3u8 master playlist.Also, play-dl always provides the best audio quality at the end of the format array for non-livestreams.We could also replace node-fetch with native-fetch when the project is upgraded to Node 18 or later.Edit: I replaced ffmpeg with play-dl stream for non-caching videos. This should actually resolve the issue mentioned in #1018. Also I don't know why the Lint check fails.