-
Notifications
You must be signed in to change notification settings - Fork 52
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
Support multiple stores natively #118
Comments
Hello, the extension doesn't have a coupling to That's why if you have multiple stores, you need to use browser extension config, but on the other hand it allows to have complete freedom, you to have multiple stores even if you use plain UPDATE: you might have hit this caveat: in order to recognize |
@maximbaz Is there anything preventing If there's nothing in the way, it might be worth us parsing that - seems like a pretty simple thing to add, and quite useful. |
|
Let's not, then. We don't want to encourage anything that would cause |
There could be an upstream tweak to accept only the first supplied store. |
Realistically I don't think it's something we'll be pursuing, but you are welcome to try to do this. We could potentially support our own environment variable, but then again I'm not really sure how much benefit is in it, if you have to configure it in one place and only one tool supports this configuration, it is far easier for people to configure it in browser extension options... |
Could browserpass potentially look at the gopass config and parse it as extra stores similar to how you would manually through the browserpass extension menu? For example, if a user has their root store at
This would provide both the mounts display name (in this case The main advantage of doing this would be for team compatibility and auto setup, it wouldn't be a lot of work to go into the browser extension on just your PC, but having to do it for every member on a team would make it a big headache when this step could potentially be automated. What are your thoughts @maximbaz ? |
Hello, thanks for bringing this up! I acknowledge that the setup shouldn't be unnecessarily complicated. I think it's important to mention some constraints that we chose for this project:
Making native host parse gopass config file would violate the first point (e.g. it would make us dependent on the gopass, if they for example choose to change config file format), and open up for requests to also support other clients (which would conflict with the third point). This makes me not super-keen on the idea... But perhaps we can still improve your workflow? What do you think about doing something on the browser side? Here's what I'm thinking: configuration in the browser extension is a simple json, stored in local storage. What if we were to add some kind of Backup/Restore functionality in the extension options? Then you could have some script that converts gopass config to browserpass config, and upload it via "Restore". Not exactly an auto-setup, but if you have a lot of stores, it would reduce the setup to a few clicks? (A certain language model is convincing me that it's possible to manipulate local storage of an extension from a python script, though I don't know if we should trust it, and whether I'd go down that hacky road even if it was technically possible 😅) |
General information
Make browserpass support multiple stores?
I wanted to know if browserpass should be supporting multiple stores too without having to add each store manually in the extension. If I set
PASSWORD_STORE_DIR
or use any store path (i.e~/.local/share/gopass/stores
), then the extension should be able to pick up secrets in all stores.https://github.com/gopasspw/gopass/blob/master/docs/features.md#multiple-stores
$/ browserpass --version
): Na.$ git describe --always
): Na.If you are getting an error immediately after opening popup, have you followed the Configure browsers documentation section?
Exact steps to reproduce the problem
Na.
What should happen?
Set
PASSWORD_STORE_DIR
or specify a multi-stores path.Browserpass to pick up secrets in all stores.
Enter any secrets.
What happened instead?
The text was updated successfully, but these errors were encountered: