-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Punycode phishing attack detection #2550
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
Comments
@anewuser ok - but if we have feature request - there's just no way to communicate those? |
@fproulx-pbox uBlock is a network blocker (filter), not URL checker. While it can protect you from known (black listed) malware domains, it is not really tasked with verifying URLs. Also, consider this:
|
@fproulx-pbox Considering what I wrote above, the solution here is simple and does not require any changes in uBlock:
|
Actually the issue has been fixed in Chrome 58 released today. |
I will consider adding a small sign (something like superscript letter |
It's a browser issue. For Firefox, Palemoon, K-meleon: |
I'm with @bershan2 here. This doesn't seem to me to be in uBO's wheelhouse since there is no filtering, so the maintenance overhead that comes with introducing a new capability along with its UI may not make sense. Besides consider: the suffix |
It will be displayed in the advanced user pane, and only for when there is a mix of ascii and unicode characters in the domain name.
I can't make sense of this. This is a rendering issue, not a new capability one. The code change is merely how to render domain names in the advanced user pane, there is no new capability introduced. Let's not turn a simple issue into a never ending bikeshedding discussion. |
Key word: mitigation. There is no magic solution for the whole issue. The chosen (and trivial codewise) change here is to render the base domain name with the label The base domain is a mix of one more of the "22 confusable unicode characters" and zero or more normal ascii characters, and nothing else. Examples:
The label is strictly informative of the above condition -- the user is responsible to act based on that information. The final solution is inspired from the discussions at:
Additionally, the punycode version of a base domain name will be provided through a tooltip if there is at least one unicode character in the base domain name minus public suffix. |
I did some more reading and modified the condition under which uBO will use the |
I'm going to need to revise slightly again, the |
When I test your IDN test site and look in advanced pane the international root domains correctly have the IDN symbols. But if you click +all to show subdomains it is not that clear. Unlike Also, try creating some dynamic filters from advanced pane for both api.jquery.com (real) and api.jquery.com (mimicry) and then go to My Rules. There are 2 rules that are almost visually identical as What do you think? |
Uh oh!
There was an error while loading. Please reload this page.
Describe the issue
Punycode based phishing attacks may become a threat to consider
There are Chrome extensions that detects those - it'd be nice to have it part of UBlock
like
https://chrome.google.com/webstore/detail/punycode-domain-detection/fkenopinnpinfcjneoanjoimhkmdcjne
or
https://chrome.google.com/webstore/detail/idn-safe/kegeenojcnijgmfgkcokknkbpmjcabdm
The text was updated successfully, but these errors were encountered: