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

Add --html=$html option to solve the "429 too many request" issue #4551

Closed
TheGr33k opened this issue Dec 23, 2014 · 8 comments
Closed

Add --html=$html option to solve the "429 too many request" issue #4551

TheGr33k opened this issue Dec 23, 2014 · 8 comments

Comments

@TheGr33k
Copy link

@TheGr33k TheGr33k commented Dec 23, 2014

I think i have though of a durable solution to solve the "429 too many request" issue.Suppose i can get the html for the page by any other means (like using my ip addresses binded to my network interface) , i can add the html as a parameter in the youtube-dl is present.If the html is present as an option , then youtube-dl won't have to fetch html once again resulting in fewer request to youtube and thus less chance of getting the error.On top of that its increased performance for youtube-dl since it has less work to do to generate the direct youtube url.It should be too hard i think

Moreover you can add the throttle feature as well to further foolproof youtube-dl it may help in the 429 issue

Check #4550

@TheGr33k
Copy link
Author

@TheGr33k TheGr33k commented Jan 9, 2015

Anyone could look at it please .....

@phihag
Copy link
Contributor

@phihag phihag commented Jan 9, 2015

This approach could not possibly work request information from multiple URLs (HTML or other). Therefore, I see two possible solutions:

  • Implement IP binding right in youtube-dl
  • Allow using external downloaders
@TheGr33k
Copy link
Author

@TheGr33k TheGr33k commented Jan 10, 2015

I don't understand why its not possible.The approach is rather simple, let me explain once again.Currently youtube-dl fetch youtube html page and then extract the direct url/download depending on options

My idea is that if you add an option so that we can feed the html to youtube-dl , youtube-dl just have to extract the direct url/download depending on options

IP binding is also awesome but since many people have asked for it and its hasn't been done yet(i assume its a tough job) the html option would allow people to mimic the ip binding by fetching the html of the page via curl using an ip address of their choice and then feeding it to youtube-dl

@jaimeMF
Copy link
Collaborator

@jaimeMF jaimeMF commented Jan 10, 2015

And you are assuming that only one webpage is needed for extracting the info, but that's not true. In the case of YouTube: we need the actual video webpage and the dash manifest (which contains some additional formats), and for age restricted videos we need an additional page.

@TheGr33k
Copy link
Author

@TheGr33k TheGr33k commented Jan 10, 2015

Yes i was assuming that only one page is needed ........Philip can you
please add the bind ip feature asap(at least for the g option).The 429 too
many request is really nagging me here

On Sat, Jan 10, 2015 at 2:07 PM, Jaime Marquínez Ferrándiz <
notifications@github.com> wrote:

And you are assuming that only one webpage is needed for extracting the
info, but that's not true. In the case of YouTube: we need the actual video
webpage and the dash manifest (which contains some additional formats), and
for age restricted videos we need an additional page.


Reply to this email directly or view it on GitHub
#4551 (comment).

@phihag phihag closed this in be4a824 Jan 10, 2015
@phihag
Copy link
Contributor

@phihag phihag commented Jan 10, 2015

Starting from youtube-dl 2015.01.10.1, you can make use of the --source-address option. See our FAQ if you need help updating.

We may implement external downloaders too, but I do think the original issue has been solved.

@phihag
Copy link
Contributor

@phihag phihag commented Jan 10, 2015

As mentioned in our contribution guidelines, please do not post about multiple different issues in a single issue entry.

Instead, create new issue entries for new issues. Do not worry about creating issues; it is free of charge (and allows us to keep track which problems have been solved and which have not been).

As also mentioned in the bug reporting guidelines, unless you are certain that it is not necessary, please include the output you get when you add -vC to your command line (using youtube-dl 2015.01.10.2 or newer which include better debugging capabilities). In your case, a pcap dump would also be extremely helpful.

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.