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

Non-ASCII character for system profile names causes problems with Brave profile #9425

Closed
srirambv opened this issue Jun 13, 2017 · 22 comments
Closed
Assignees
Labels
bug fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. l10n needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. OS/Windows priority/P2 Crashes. Loss of data. Severe memory leak. windows-important wontfix

Comments

@srirambv
Copy link
Collaborator

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Non-ASCII character for system profile names causes problems with Brave profile

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10

  • Brave Version (revision SHA):
    Brave 0.16.9
    rev fba98fc
    Muon 4.0.3

  • Steps to reproduce:

    1. Create a Windows User account with non-ASCII characters
    2. Install Brave, Bave doesn't get listed as one of the default browsers
    3. Extensions don't get listed and even if enabled doesnt show the preferences page
  • Actual result:
    Non-ASCII character for system profile names causes problems with Brave profile

  • Expected result:
    Brave should be installed properly irrespective of how the user account name is created

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes, Confirmed by @jonathansampson

  • Is this an issue in the currently released version?
    Yes

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:

  • Any related issues:
    https://community.brave.com/t/cant-access-lastpass-account-preferences/3518/11
    cc: @bbondy @jonathansampson

@srirambv srirambv added this to the 0.17.x (Beta Channel) milestone Jun 13, 2017
@jonathansampson
Copy link
Collaborator

I created a Windows user named Şąሐየ፴ọⴷ. I then signed-in, and downloaded Brave.

Upon loading Brave, I found a broken shortcut for BraveDefaults.exe on my desktop. Once Brave loaded -up, it asked me to set my default browser. Windows did not list Brave as one of the options.

In Brave, I navigated to about:preferences#extensions, and enabled LastPass. LastPass never became enabled, and was never listed in about:extensions.

I suspect there are more issues, but this was the tip of the iceberg.

@jonathansampson
Copy link
Collaborator

cc @bsclifton, who also had thoughts about what may be to blame.

@luixxiul luixxiul added the l10n label Jun 14, 2017
@savehansson
Copy link

Nice to see that my 30+ years of experience as a software developer pays off ;)

Kudos to you for your handling of this issue both at https://community.brave.com/t/cant-access-lastpass-account-preferences/3518/11 and here.

By the way, in my case the profile name was Håkan, so many fairly common usernames (Windows-1252) are affected, not only Chinese, Arabic etc.

@bsclifton
Copy link
Member

bsclifton commented Jun 15, 2017

Since this has been an issue for a while (I also saw similar reported issues for Electron), I'm going to remove the milestone. I do agree this is high priority though! 😄 We can probably identify a good amount of impact (and possibly some of the root cause) by looking at the open issue log for Electron and sharing issues here

@bsclifton bsclifton removed this from the 0.17.x (Beta Channel) milestone Jun 15, 2017
@bsclifton
Copy link
Member

Bumping down to impact medium since most Brave users are likely not running into this

I'm if there's any impact on macOS or Linux? or if this is Windows only (which it sounds like)

@jonathansampson
Copy link
Collaborator

jonathansampson commented Sep 13, 2017

+1 from Twitter: https://twitter.com/BraveSampson/status/907637719282700290

The user's name is Sébastien. The é alone, I believe, is enough to prevent much of Brave's core functionality.

@jonathansampson
Copy link
Collaborator

jonathansampson commented Sep 13, 2017

@bsclifton Do you have a link to the Electron issue? I came across a couple commits that appear to be addressing similar issues:

I think we should consider putting this on a milestone, and addressing it a.s.a.p. as it potentially alienates large groups of international users. And when Brave fails in this space, it fails catastrophically and silently, giving no indication as to what was the cause.

@tdeinha
Copy link

tdeinha commented Nov 14, 2017

+1. Brazilian name with "é" which is pretty common and without the extensions Brave becomes unusable for me.

@bsclifton bsclifton added priority/P2 Crashes. Loss of data. Severe memory leak. and removed impact/medium labels Nov 22, 2017
@bsclifton bsclifton added this to the Backlog (Prioritized) milestone Nov 22, 2017
@jonathansampson
Copy link
Collaborator

May be related to #5369.

@mzKas
Copy link

mzKas commented Dec 6, 2017

Got the same problem with my name "Luís" on Windows profile name.
Have tried on another machine with just "Luis" and it works fine...
Any resolution on this?

@jonathansampson
Copy link
Collaborator

@mzKas No fix at this time, but we will have this addressed in a future release. Apologies for the inconvenience.

@glenninja
Copy link

Brave browser installs as default but extensions dont work.

@mzKas
Copy link

mzKas commented Jan 16, 2018

Still waiting for a fix for this....
Its unbelievable that you want a stable browser and you have a bug that none of the other chromium based browsers have...

@savehansson
Copy link

I agree @mzKas. This should be easy to fix. Low hanging fruit!

@mzKas
Copy link

mzKas commented Jan 24, 2018

aaaaaaaaand guess what?
nothing,..
im about go give up on this browser because of a stupid bug that doesnt exist elsewhere...

@bsclifton
Copy link
Member

bsclifton commented Jan 24, 2018

@mzKas @savehansson apologies this hasn't been fixed yet ☹️ It is definitely an important one to fix.

I don't think it's low hanging fruit though- I am suspecting this needs a patch in the Muon level, like the one @jonathansampson linked to:
electron/electron@8633829

I'll share with the team and see if there's anything we can do to expedite this

@bsclifton bsclifton modified the milestones: Backlog (Prioritized), 0.22.x (Nightly Channel) Jan 24, 2018
@bsclifton
Copy link
Member

@mzKas @savehansson can you describe the impact in more detail?

Are you just not able to set default browser / configure extensions? Or is nothing being saved at all (bookmarks, etc)? How bad is it?

@bsclifton bsclifton added the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Jan 25, 2018
@mzKas
Copy link

mzKas commented Jan 29, 2018

@bsclifton I have uninstalled it and wont be trying it for a while...
I remember i had trouble setting it as "not-default" because it kept asking for it so it wasnt saving that particular option.

@bbondy bbondy modified the milestones: 0.22.x (Developer Channel), 0.23.x (Nightly Channel) Feb 25, 2018
@bsclifton bsclifton modified the milestones: 0.23.x (Nightly Channel), 0.24.x, Completed work, Triage Backlog, Backlog (Prioritized) Feb 26, 2018
@alexwykoff alexwykoff modified the milestones: Backlog (Prioritized), 0.25.x (coming soon) May 1, 2018
@Soryathena
Copy link

So, I installed Brave earlier coz it sounds like a cool idea. I can buy into that. So... I've spent the last couple of hours trying to figure out why none of the extensions work (only three are even installed). I can see now it's this. It's not only an issue for international users -- my name is Zoë. I'm a native English speaker from the UK. I'm used to my name causing issues with stuff online, but I didn't expect it to break my browser, lol.

Anyways I'd really like to support this project. I'm not enthusiastic enough to scrap my entire Windows profile over it. I see it's been a known issue for like a year now so I hope it can be fixed soon.

@SilverPuppy
Copy link

SilverPuppy commented Jun 8, 2018

I have been working around bugs to support this project for well over a year now. I have provided constructive criticism and feedback; I wish I could provide more, but I just lack the code skills.

This project is being undertaken by a dedicated team who really are trying to do some amazing things, which I believe will shape internet history in a great way. This is growing pains. :D My suggestion would be to open it up once in awhile just to check for updates, and hopefully soon it will get an update that makes it work as intended.

I will say that there is a workaround for this particular issue, which is simply to turn off the option to view PDFs in the browser. This will give you a download which can then be opened in an external viewer. I have been using this solution to work around another bug which I would hope can be resolved soon. However, that assumes that PDF.js is the only extension you would ever actually use, as it is in my case.

@jonathansampson
Copy link
Collaborator

@SilverPuppy This issue impacts more than just PDF viewing. The lack of support for non-ASCII characters results in Brave having no ability to remember which extensions are enabled, and many other issues. See my comment here: #9425 (comment)

@bsclifton bsclifton added wontfix fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. labels Jul 26, 2018
@bsclifton bsclifton removed this from the 0.25.x (Nightly Channel) milestone Jul 26, 2018
@bsclifton
Copy link
Member

Unfortunately, this is not an issue we'll be able to fix before moving to Brave Core. However, once we do make the move, this bug will be fixed 😄 (it's not a problem w/ Brave Core- only in our Muon codebase)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. l10n needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. OS/Windows priority/P2 Crashes. Loss of data. Severe memory leak. windows-important wontfix
Projects
None yet
Development

No branches or pull requests