-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
Reading from pipes #174
Comments
This used to be a feature of pass-import, I removed it long time ago as it was causing issues for some pm (and because nobody was using it). The good news it that pass-import still has the logic to support it. So PR are more than welcome. You should hack around the If you get an issue with format detection you can force it to use a given import class with Note: direct import using the |
Can you perhaps point me to the ticket/PR for the problems you had that made you want to take this out? Ideally, I'd try to sidestep those.
Hmm, maybe I should concentrate on helping on this, then! Where can I join the fun? |
I've since concentrated all my password on Bitwarden, and I've been happy enough using it. But I'd be interested to know when your new |
You can now try 7b7bb4d from the dev branch (#180). But please note, this is still in dev stage. I usually do not commit such code that is not fully tested in CI (it should still works fine for basic use), but as I do not have time to work on this these days, it seems fair to allow you to play with it anyway. To try it, from the dev branch: |
Thank you for sharing! 🙌 However, I'm at a point in my life where other obligations prevent playing with this from bubbling up in my priority stack. I'll let you know if I happen to get a chance, tho! |
For future me: this feature is currently not a priority. |
I'd like to add something like
bw --raw export --format json | pass import -fa bitwarden -
to my cron.However, it seems like pass-import always wants a file to read from, and not a pipe. Is this something you'd be open to get a PR for? Where's the right place to make this change, roughly?
The text was updated successfully, but these errors were encountered: