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

CGI flag interpretation not order-agnostic #304

Closed
srunni opened this issue Mar 8, 2012 · 3 comments
Closed

CGI flag interpretation not order-agnostic #304

srunni opened this issue Mar 8, 2012 · 3 comments

Comments

@srunni
Copy link

@srunni srunni commented Mar 8, 2012

When the CGI flags aren't ordered so that v is first, youtube-dl fails to download the video.

For example, try this link, as opposed to this one. Both load correctly in a browser, but only the second can be downloaded by youtube-dl.

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Mar 8, 2012

Perfectly working on my side.

$ ./youtube-dl "http://www.youtube.com/watch?feature=player_embedded&v=xLYiIBCN9ec"
[youtube] Setting language
[youtube] xLYiIBCN9ec: Downloading video webpage
[youtube] xLYiIBCN9ec: Downloading video info webpage
[youtube] xLYiIBCN9ec: Extracting video information
[download] Resuming download at byte 3697985
[download] Destination: xLYiIBCN9ec.mp4
[download]   3.2% of 140.18M at  713.58k/s ETA 03:14^C
ERROR: Interrupted by user
$ ./youtube-dl "http://www.youtube.com/watch?v=xLYiIBCN9ec"
[youtube] Setting language
[youtube] xLYiIBCN9ec: Downloading video webpage
[youtube] xLYiIBCN9ec: Downloading video info webpage
[youtube] xLYiIBCN9ec: Extracting video information
[download] Resuming download at byte 4745537
[download] Destination: xLYiIBCN9ec.mp4
[download]   4.5% of 140.18M at  739.06k/s ETA 03:05^C
ERROR: Interrupted by user

Please be sure to have updated ./youtube-dl -U and note that the & is a special symbol on the shell, so you have to put the url in ""

@srunni
Copy link
Author

@srunni srunni commented Mar 9, 2012

It was the & symbol - sorry about that!

@FiloSottile
Copy link
Collaborator

@FiloSottile FiloSottile commented Mar 9, 2012

Don't worry :)
Please close the issue as I have not the rights to do so.

@phihag phihag closed this Mar 9, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.