Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

multibyte characters should be encoded #5220

Closed
luixxiul opened this issue Oct 28, 2016 · 11 comments
Closed

multibyte characters should be encoded #5220

luixxiul opened this issue Oct 28, 2016 · 11 comments
Labels
fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. l10n wontfix

Comments

@luixxiul
Copy link
Contributor

luixxiul commented Oct 28, 2016

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.

screenshot 2016-10-28 15 45 50

the above is Brave, the below is Chrome.

screenshot 2016-10-28 15 54 52

@luixxiul luixxiul added the l10n label Oct 28, 2016
@luixxiul luixxiul added this to the 1.1.0 milestone Oct 28, 2016
@Sh1d0w Sh1d0w changed the title multibyte characters should be encorded multibyte characters should be encoded Oct 28, 2016
@luixxiul
Copy link
Contributor Author

luixxiul commented Nov 16, 2016

You cannot input Japanese characters in "Home Page" setting on about:preferences as well

screenshot 2016-11-16 21 32 36

@luixxiul
Copy link
Contributor Author

luixxiul commented Jun 18, 2017

On URL bar (macOS) multibyte characters are encoded well.

screenshot 2017-06-19 2 14 20

Maybe thanks to bloodhound update? #8824

CC @darkdh for confirmation with Chinese characters.

Edit:
Brave 0.19.0
rev c5968c5

@darkdh
Copy link
Member

darkdh commented Jun 18, 2017

screen shot 2017-06-18 at 2 25 54 pm

screen shot 2017-06-18 at 2 26 21 pm

and your example will give me

screen shot 2017-06-18 at 2 27 15 pm

I'm on 0.17.3 MacOS
cc @diracdeltas for opinions

@bsclifton
Copy link
Member

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)

@diracdeltas
Copy link
Member

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

@luixxiul
Copy link
Contributor Author

As long as we stick to the Chromium way, it looks good to me.

http://xn--p8j9a0d9c9a.xn--q9jyb4c/index.html looks just simply corrupted, and I think we should avoid that.

@diracdeltas
Copy link
Member

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)

@kenorb
Copy link
Contributor

kenorb commented Aug 25, 2017

Same for this URL:

I see %E1%9A%AB... instead of the Unicode characters.

  • Expected result:

I could see the URL similar as in Chrome or Firefox, e.g.

screen shot 2017-08-25 at 23 52 34

@alexwykoff alexwykoff modified the milestones: 1.1.0, Backlog Nov 1, 2017
@bsclifton bsclifton removed this from the Triage Backlog milestone Sep 1, 2018
@bsclifton bsclifton added wontfix fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. labels Sep 1, 2018
@bsclifton
Copy link
Member

@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?

@diracdeltas
Copy link
Member

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

@bsclifton
Copy link
Member

@diracdeltas created brave/brave-browser#1004 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. l10n wontfix
Projects
None yet
Development

No branches or pull requests

6 participants