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 a --dns-resolver option #4726

Open
TheGr33k opened this issue Jan 16, 2015 · 4 comments
Open

Add a --dns-resolver option #4726

TheGr33k opened this issue Jan 16, 2015 · 4 comments
Labels

Comments

@TheGr33k
Copy link

@TheGr33k TheGr33k commented Jan 16, 2015

It would be nice to have a feature that allows you to state the nameserver you want youtube-dl to use to resolve domain name.Some common example include OpenDNS , Google Public dns etc....

Regards

@jaimeMF jaimeMF added the request label Jan 16, 2015
@Kaligule
Copy link

@Kaligule Kaligule commented Jan 22, 2015

I think you should be able to achive the same effect by using nslookup and a |.

@TheGr33k
Copy link
Author

@TheGr33k TheGr33k commented Jan 22, 2015

@Kaligule Could you please show an example ?

Even then we won't be able to know for sure what dns server youtube-dl sees

Feel free to correct me if i am wrong

On Thu, Jan 22, 2015 at 8:54 PM, Kaligule notifications@github.com wrote:

I think you should be able to achive the same effect by using nslookup
and a |.


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

@phihag
Copy link
Contributor

@phihag phihag commented Jan 23, 2015

The main problem is not that we cannot hook up a custom resolver class, but that we need to actually implement (a simple subset of) DNS. Unless someone contributes the necessary code (basically a pure Python implementation of gethostbyaddr), this issue will not be resolved.

@yan12125
Copy link
Collaborator

@yan12125 yan12125 commented Sep 23, 2016

For those who are interested in it: I have a proof-of-concept branch at https://github.com/yan12125/youtube-dl/tree/custom-dns, which uses the ISC licensed dnspython.

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
5 participants
You can’t perform that action at this time.