-
-
Notifications
You must be signed in to change notification settings - Fork 156
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
tosiewytnie.pl #55
Comments
Thanks @xxcriticxx added to false positives on https://github.com/mitchellkrogza/CENTRAL-REPO.Dead.Inactive.Whitelisted.Domains.For.Hosts.Projects and will be stripped out on next build. Keep em coming 👍 |
@xxcriticxx @funilrys if either of you can think of any domains which should be permanently whitelisted always, please review the list here and either send a PR with more domains to be whitelisted or simply list them in Issues on that repo and I will add them. |
@mitchellkrogza I didn't look into the code but do you use regex match? Cause for example: It'll save us some bytes to have
I don't know what you think about it but I think it can be great assuming that only legitimate (big company or well-known website) domains are on that whitelist. |
I agree totally, haven't figured out a regex in my cleaning functions to handle that. Much better to have just root domains and then *.domain.com get's stripped out of hosts lists. Do you want to review those list cleaning functions for me ??? You're smarter with Regex than me 😁 |
Thanks @xxcriticxx those have now all been whitelisted in https://github.com/mitchellkrogza/CENTRAL-REPO.Dead.Inactive.Whitelisted.Domains.For.Hosts.Projects/commit/047231eb58dc03149753fb7f8146d1de939047a2 |
@xxcriticxx we still have about another 5 days of dead domain testing to go before fresh links get pushed out again. |
@mitchellkrogza no problem! |
@xxcriticxx fresh raw links pushed out today. Continuation of dead domains testing continuing in a separate environment. |
::: /etc/pihole/list.41.hosts.ubuntu101.co.za.domains (1 results)
0.0.0.0 tosiewytnie.pl
The text was updated successfully, but these errors were encountered: