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

Adding a utility to help with managing lists #4

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wttw
Copy link

@wttw wttw commented Aug 19, 2022

I found some sites I use being blocked and it wasn't easy to work out why.

This PR adds a small utility donut that can fetch all the blocklists in the default set, then search them to see why a hostname is blocked (e.g. it's on line X of the blocklist retrieved from URL Y).

It doesn't share any code with donutdns proper - there are one or two things it should share, like sources.json, but I copied them instead. So it could live in it's own repo, but it seemed to make more sense for it to be part of this one.

@shoenig
Copy link
Owner

shoenig commented Oct 16, 2022

Hi @wttw, this is awesome! Let me apologize for taking so long to respond. This is a feature I wanted pretty much since day 1, but in the back of my mind it felt like adding support for subcommands would be the way to go. But to make that work a bunch of refactoring needed to happen to make the list downloading stuff re-usable. In #16 we now have a rudimentary donutdns check subcommand - if you're still interested it'd be great to expand it with the details and caching behavior you've got in this PR.

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