-
-
Notifications
You must be signed in to change notification settings - Fork 155
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
pastebin.com #9
Labels
false positive
all false positive review requests
Comments
👍 |
mitchellkrogza
added a commit
that referenced
this issue
Aug 12, 2017
Fixed in fcaa791 |
mitchellkrogza
added a commit
that referenced
this issue
Aug 14, 2017
mitchellkrogza
added a commit
that referenced
this issue
Aug 16, 2017
- Fixes #1 #2 #3 #4 #5 #7 #8 #9 Former-commit-id: d73ecb3b32cb45842d1c5e16cd2063a29e8a3c82 [formerly b002774] Former-commit-id: beb4275ce44d4ec88f886a4dea40b3233de3965e Former-commit-id: 12aa262e65cee617ac3ba9c6792f4bc8d2f7aa6d Former-commit-id: e2af485 Former-commit-id: 2599083b1298af9301138685f7fce35bee95e208
@xxcriticxx I should actually remove these as the way hosts works is it see's the domain name and ignores the /whatever after it. So that would mean adding pastebin.com all by itself which is not a good idea. That's what caused you the money.cnn.com issue. It had money.cnn.com/something-else-after-it and hosts uses DNS (very simply too) and it just sees money.cnn.com and blocks the whole domain. |
mitchellkrogza
added a commit
that referenced
this issue
Aug 21, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
::: /etc/pihole/list.41.hosts.ubuntu101.co.za.domains (3 results)
ALL: pastebin.com/h0aj4gel
ALL: pastebin.com/n4anjbnw
ALL: pastebin.com/qhqa0mxn
The text was updated successfully, but these errors were encountered: