-
Notifications
You must be signed in to change notification settings - Fork 1
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
Overzealous filters #16
Comments
I believe there was another facebook TLD, and yes, we should probably just block TLDs in the default filters. Making the Regex more complicated is something that should be avoided. |
In other words, we should have two separate filters, I agree making the regex more complicated is not a good idea. Perhaps we On 06/30/2016 09:45 AM, Joshua Bemenderfer wrote:
|
*facebook.com would match that, right? And yes. I think facebook's like buttons or ad thingies are on a different TLD. Still can't find it though. :/ |
Yes, it would match both Looks like it's |
Ah, that was it. |
The default filters block
https://facebook.github.io/
. The filter responsible is*facebook.*
I think there ought to be a way to match the main domain and all sub domains without blocking domains that end with the same name, but I can't think of what the right syntax would be. Perhaps something like*.?facebook.*
Seems that could be problematic.For this particular rule, would there by any problem changing it to
*facebook.com
? That would still block domains that end withfacebook.com
of course. Are there any Facebook TLDs that need to be blocked, other than.com
, that would make this not work?The text was updated successfully, but these errors were encountered: