starred/youtube-dl

Fork 0

mirror of https://github.com/ytdl-org/youtube-dl.git synced 2026-03-02 23:07:04 -05:00

Downloading wrong playlist [bug] #519

New issue

Closed

opened 2026-02-20 21:15:43 -05:00 by deekerman · 6 comments

deekerman commented

2026-02-20 21:15:43 -05:00

Owner

Originally created by @kevin-hanselman on GitHub (Jan 31, 2013).

Originally assigned to: @FiloSottile on GitHub.

I am trying to downloading this playlist:

http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

I have tried:

youtube-dl http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

youtube-dl PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

youtube-dl http://www.youtube.com/view_play_list?p=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

All of them start downloading a 100 video playlist that starts with Gangnam Style...

I have tried downloading a few other playlists and they work just fine, which leads me to believe there is a potential bug in the program. If not, am I just doing something wrong?

Originally created by @kevin-hanselman on GitHub (Jan 31, 2013). Originally assigned to: @FiloSottile on GitHub. I am trying to downloading this playlist: http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0 I have tried: `youtube-dl http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0` `youtube-dl PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0` `youtube-dl http://www.youtube.com/view_play_list?p=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0` All of them start downloading a 100 video playlist that starts with Gangnam Style... I have tried downloading a few other playlists and they work just fine, which leads me to believe there is a potential bug in the program. If not, am I just doing something wrong?

deekerman closed this issue

2026-02-20 21:15:45 -05:00

deekerman commented

2026-02-20 21:15:47 -05:00

Author

Owner

@FiloSottile commented on GitHub (Feb 3, 2013):

Hm, it is a bug related to the PL prefix that we strip.

Here we have to include it; sometimes we must not include it; sometimes including it changes the page layout. Great.

@FiloSottile commented on GitHub (Feb 3, 2013): Hm, it is a bug related to the `PL` prefix that we strip. Here we have to include it; sometimes we must not include it; sometimes including it changes the page layout. Great.

deekerman commented

2026-02-20 21:15:47 -05:00

Author

Owner

@phihag commented on GitHub (Mar 3, 2013):

The playlist you referenced has been modified significantly. Do you have a new testcase for this problem? Should be solved anyways though.

@phihag commented on GitHub (Mar 3, 2013): The [playlist you referenced](http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0) has been modified significantly. Do you have a new testcase for this problem? Should be solved anyways though.

deekerman commented

2026-02-20 21:15:47 -05:00

Author

Owner

@kevin-hanselman commented on GitHub (Mar 3, 2013):

@FiloSottile commited a fix to my issue. I cleared my playlist because I successfully downloaded those videos.

@kevin-hanselman commented on GitHub (Mar 3, 2013): @FiloSottile commited a fix to my issue. I cleared my playlist because I successfully downloaded those videos.

deekerman commented

2026-02-20 21:15:48 -05:00

Author

Owner

@FiloSottile commented on GitHub (Mar 3, 2013):

The issue was not related to playlist content, but to the fact that the web
interface required the PL prefix, while the API eats it also without.

The official docs say that it is mandatory now, so (I think) that I edited
the regex to include it.

@FiloSottile commented on GitHub (Mar 3, 2013): The issue was not related to playlist content, but to the fact that the web interface required the PL prefix, while the API eats it also without. The official docs say that it is mandatory now, so (I think) that I edited the regex to include it.

deekerman commented

2026-02-20 21:15:48 -05:00

Author

Owner

@FiloSottile commented on GitHub (Mar 4, 2013):

Note: that playlist is still a good test target, have a look at
http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0
and
http://www.youtube.com/playlist?list=MCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

@FiloSottile commented on GitHub (Mar 4, 2013): Note: that playlist is still a good test target, have a look at http://www.youtube.com/playlist?list=PLMCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0 and http://www.youtube.com/playlist?list=MCmkNmxw6Z9eduM7BZjSEh7HiU543Ig0

deekerman commented

2026-02-20 21:15:48 -05:00

Author

Owner

@kevin-hanselman commented on GitHub (Mar 4, 2013):

That's a long random character string to get matched to YouTube Daily Top Tracks playlist. What are the chances? 😃

@kevin-hanselman commented on GitHub (Mar 4, 2013): That's a long random character string to get matched to YouTube Daily Top Tracks playlist. What are the chances? :smiley:

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/youtube-dl#519

No description provided.

Rows
Columns