browserpass / browserpass-legacy

Legacy Browserpass repo, development is now happening at:
https://github.com/browserpass/browserpass-extension
MIT License
1k stars 80 forks source link

Add instructions for installing on Windows through WSL #193

Closed jtsymon closed 6 years ago

jtsymon commented 6 years ago

Moving instructions for working with WSL to README.md. Some stuff to look at:

jtsymon commented 6 years ago

Apparently the PR title doesn't link issues 😞

192

maximbaz commented 6 years ago

Thanks!

  1. Extending the install.ps1 sounds like a good idea, do you want to try implementing it? I won't be able to test this bit personally, but we have a couple of Windows users, I can ping them.
  2. Yeah I don't think it's a good idea to implement pin-entry dialog in the extension itself, don't want to touch users' master passwords 🙂 But it is a very important part, or people will be confused why nothing is working. Could you maybe mention this in the README as well? Just mention that unfortunately there is such issue, and as a workaround it is required to manually unlock the key, e.g. just show a password inside WSL.
maximbaz commented 6 years ago

By the way, how come on WSL both windows and linux host applications are needed? Are they both being used? The .bat file only references the linux host app...

jtsymon commented 6 years ago

The Windows host application isn't actually needed, but the config files and registry keys are used for the browser to find the host application. Normally it'd work like Browser -> Native Messaging config -> Host Application But this does Browser -> Native Messaging config (Windows) -> Host Application (Linux) So we need the config files from the Windows host application (but not the actual host application), and the Linux host application (but not its config files).

maximbaz commented 6 years ago

Huh, makes sense now, thanks!

jtsymon commented 6 years ago

Oh I just noticed another problem. My original .bat file assumes the Linux host application is in ~/browserpass/, but that actually depends on where the user put the browserpass files when installing the Linux host application. So the .bat file installed with the Windows host application doesn't actually know where to find the Linux host application. This could be handled by having install.ps1 also install the Linux version under WSL (behind a prompt or flag), but that could get pretty messy. The cleanest option is probably a separate "WSL" zip that contains the Windows config with the Linux host application and a different install script, but that creates extra maintenance work for very few users. ... so maybe we should scrap the "automatically create the .bat file" idea, and just say "create a batch file that uses WSL to load the Linux browserpass host application."

maximbaz commented 6 years ago

Agreed, let's go with a simple option then. If there will be a public demand for automating then, we will revisit :wink:

Add the description of the problem with password-protected key and a known workaround, and let's merge it.

jtsymon commented 6 years ago

I added the note about password-protected keys, and also added a note about why to use this (hopefully keeps people from accidentally using this when they just want a Windows setup).

jtsymon commented 6 years ago

Only problem is now its taking up half the README for an edge case setup 😆

maximbaz commented 6 years ago

That is totally fine, and I don't see it as an edge case, I like the idea behind WSL and I hope it gets better / more widespread in the future 😉

maximbaz commented 6 years ago

Cool, thanks a lot - merging in a moment.