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 Quad9 dns configuration #10

Merged
merged 1 commit into from
Oct 24, 2020
Merged

Add Quad9 dns configuration #10

merged 1 commit into from
Oct 24, 2020

Conversation

aalmenar
Copy link

This add another DNS service to the default list.

If you want i can make a pull request with the most used ones like, opendns and so on. Also to include cloudflare for families and other from quad9, so the user can select most of them and save him from entering a lot of information by hand.

@kkebo
Copy link
Owner

kkebo commented Oct 16, 2020

Thank you for your pull request.

Which DNS services do you think should be added to the default list and why? If the default list become long, I think I need to improve the sidebar UI and add test code for each DNS service's validation.

@aalmenar
Copy link
Author

Which DNS services do you think should be added to the default list and why? If the default list become long, I think I need to improve the sidebar UI and add test code for each DNS service's validation.

I would add quad9, opendns and some people may want to use NextDNS. Those are the ones I may like to include. But as you said maybe the ui needs some reworking if the list becomes large.

@kkebo
Copy link
Owner

kkebo commented Oct 19, 2020

OK, thanks.

I plan to add something like "Create from Template" here, and add Quad9, OpenDNS, NextDNS, CloudFlare, and Google Public DNS to it. Then, I'll remove all the current default list.
image

How about that?

I'll merge this pull request in the meantime after testing servers.

@kkebo kkebo merged commit 89bf37d into kkebo:main Oct 24, 2020
@comzeradd comzeradd mentioned this pull request Nov 3, 2020
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