Skip to content
This repository has been archived by the owner on Dec 3, 2023. It is now read-only.

Won't recognize playlists as such #271

Open
DiegoFleitas opened this issue Dec 1, 2019 · 9 comments
Open

Won't recognize playlists as such #271

DiegoFleitas opened this issue Dec 1, 2019 · 9 comments
Labels

Comments

@DiegoFleitas
Copy link
Contributor

It does work when the link contains the playlist keyword
https://www.youtube.com/playlist?list=PLzIUZKHPb1HbqsPMIFdE0My54iektZrNU
but there is an alternative way to building a playlist url
https://www.youtube.com/watch?v=shF8Sv-OswM&list=PLzIUZKHPb1HbqsPMIFdE0My54iektZrNU

@dhiahassen
Copy link

noticed that too !

@yafp
Copy link

yafp commented Feb 15, 2020

@DiegoFleitas
Is that a problem just with node-youtube-dl or youtube-dl itself as well?
Regarding node-youtube-dl: have you been using .exec or the default way?

@DiegoFleitas
Copy link
Contributor Author

@yafp as far as I know it only happens with node-youtube-dl, since youtube-dl from the command line works perfectly for me. I'm just modifying the string to make it work but I think it should be treated as a bug in here

@Kikobeats Kikobeats added the bug label Feb 16, 2020
@Kikobeats
Copy link
Collaborator

Could be a problem related to how the url is interpreted since the - can be mixed with flags parser 🤔

@xylish7
Copy link

xylish7 commented Jul 23, 2020

Any news on this ? I have the same problem and it's a pretty important feature to be able to download playlists.

@DiegoFleitas
Copy link
Contributor Author

@xylish7 its a pretty easy fix, just detect the list parameter and transform your unsupported url into a supported one with the playlist keyword if you need it quick as my example above. Note the playlist id is still the same

@Kikobeats
Copy link
Collaborator

@DiegoFleitas can you add the example to README? 🙂

@DiegoFleitas
Copy link
Contributor Author

Sure, but I think a proper fix would be best since my "fix" is actually to circumvent the bug.
Here's the PR https://github.com/przemyslawpluta/node-youtube-dl/pull/316/files

@xylish7
Copy link

xylish7 commented Jul 26, 2020

@DiegoFleitas I've already done this, but I thought it was nice to do this without any custom things, by fixing the bug. So that's why I was asking when this would be fixed, but thank you for your fast answer 😁

Kikobeats added a commit that referenced this issue Aug 18, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

5 participants