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

Fixed youtube code due to recent changes in the site #35

Closed
wants to merge 1 commit into from
Closed

Fixed youtube code due to recent changes in the site #35

wants to merge 1 commit into from

Conversation

yaplik
Copy link

@yaplik yaplik commented Dec 9, 2010

Youtube no longer provides www.youtube.com/get_video interface and responds on these requests with HTTP 204 No Content. Native youtube flashplayer gets HTTP 204 code as well.

Video url is now extracted from fmt_map variable.

Youtube no longer provides www.youtube.com/get_video interface and
responds on these requests with HTTP 204 No Content. Native youtube
flashplayer gets HTTP 204 code as well.

Video url is now extracted from fmt_map variable.
@rg3
Copy link
Collaborator

rg3 commented Dec 9, 2010

Won't merge this exact code. This crashes if the user requests, using -f, a format which is not in the format list.

@yaplik
Copy link
Author

yaplik commented Dec 9, 2010

True. Glad to help anyway.

@rg3
Copy link
Collaborator

rg3 commented Dec 9, 2010

Your code was indeed invaluable in getting this fixed pretty fast. Thanks a lot for taking the time to analyze the problem and providing a solution I just had to modify slightly.

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants