-
-
Notifications
You must be signed in to change notification settings - Fork 87
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
Problems importing from 1password #23
Comments
Hi, could you give me more information in order to help you.
I could add the support for |
That included all the fields managed by 1password, but not custom ones. For example, for website with security questions I add custom fields with the answers. For example, if they ask me about my dream job I have a field like |
So its seem guys at 1password are not able to have consistent csv header file. This header is completely different from the reference header used to test the import. See https://github.com/roddhjav/pass-import/blob/master/tests/db/1password.csv Nevertheless, you are right |
I ended up using kdbx (keepass) instead and I wrote a quick-and-dirty conversion script to convert from opvault to kdbx. |
I've begun working on implementing this. https://github.com/cfeckardt/pass-import/tree/onepasswordpif I'll try to finish it this week. Don't judge my code too harshly, im not a python dev. It'll be cleaned up and niceified before a PR comes. |
I had a look at your implementation, it seems to be a good start, do not hesitate to make a PR when you think its ready. |
Fixed with pass-tomb 2.2 |
Hi,
I've run into some issues importing from 1 password:
Password
and notpassword
, so the checker fails. this seems very easy to fix{(\ntag\n)}
and it ends up pretty weird in the imported entriesBut the main problem I have is with custom data. I have some entries in 1password with custom fields and they are not exported in the CSV (since they are freeform). I think the importer should use 1pif files instead of the CSV because they contain all data available and it looks easy to parse.
The text was updated successfully, but these errors were encountered: