-
Notifications
You must be signed in to change notification settings - Fork 973
multibyte characters should be encoded #5220
Comments
and your example will give me
I'm on 0.17.3 MacOS |
Good call on cc'ing @diracdeltas; because unicode characters can be used to spoof popular sites, it's probably better that we show the raw ASCII name (ex: the punycode encoded domain name). It would be nice though to either have an option (opt-in) to enable punycode decoding of the domain names (showing in the native unicode) OR for us to consider what type of UX would be needed to ensure users don't get spoofed (do we check against a list? etc) |
We display punycode-only to prevent homograph attacks, see for instance https://thehackernews.com/2017/04/unicode-Punycode-phishing-attack.html. Google has some standards for what unicode characters should be displayed as punycode. https://www.chromium.org/developers/design-documents/idn-in-google-chrome |
As long as we stick to the Chromium way, it looks good to me.
|
chromium doesn't necessarily filter out all characters that can be used for phishing (which is why the recent IDN attacks affected Chrome but not Brave) |
Same for this URL: I see %E1%9A%AB... instead of the Unicode characters.
I could see the URL similar as in Chrome or Firefox, e.g. |
@diracdeltas this bug is fixed with brave-core- but you make a good case for showing the punycode encoded domain. Should we open an issue to also do this in brave-core? |
i think it'd be good to have a tracking issue for this though we may decide it's not worth implementing in brave-core |
@diracdeltas created brave/brave-browser#1004 👍 |
Describe the issue you encountered: multibyte characters should be encoded on the URL bar and the status bar, as well as in the site listed on
about:preferences#payments
.the above is Brave, the below is Chrome.
Platform (Win7, 8, 10? macOS? Linux distro?): OS X
Brave Version: master branch
Steps to reproduce:
Any related issues:
The text was updated successfully, but these errors were encountered: