Skip to content
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

SecureSafe: Import Fail #10786

Open
1 task done
Diiieeegooo opened this issue Aug 29, 2024 · 10 comments · May be fixed by #10875 or #11402
Open
1 task done

SecureSafe: Import Fail #10786

Diiieeegooo opened this issue Aug 29, 2024 · 10 comments · May be fixed by #10875 or #11402
Labels
browser Browser Extension bug cli CLI Application desktop Desktop Application good first issue web

Comments

@Diiieeegooo
Copy link

Diiieeegooo commented Aug 29, 2024

Steps To Reproduce

  1. Go to 'Datei, Datei Import '
  2. Scroll down to 'Secure save' in Dateiformat
  3. choose your export from Secure save with comma delimeter
  4. Click on Datei Importieren
    6: you will get: Importfail: Die Daten sind nicht richtig formatiert. Kontrolliere bitte deine Import-Datei und versuche es erneut.

The Headline of the secure safe doc is "/Titel/,/Benutzername/,/Passwort/,/URL/,/Kommentar/

Expected Result

Import my passwords from secure safe

Actual Result

Error:
Die Daten sind nicht richtig formatiert. Kontrolliere bitte deine Import-Datei und versuche es erneut.

Screenshots or Videos

No response

Additional Context

No response

Operating System

Windows

Operating System Version

windows 11

Installation method

Direct Download (from bitwarden.com)

Build Version

newest one

Issue Tracking Info

  • I understand that work is tracked outside of GitHub. A PR will be linked to this issue should one be opened to address it, but Bitwarden doesn't use fields like "assigned", "milestone", or "project" to track progress.
@Diiieeegooo Diiieeegooo added bug desktop Desktop Application labels Aug 29, 2024
@djsmith85
Copy link
Contributor

Hi @Diiieeegooo and thank you for your report.

Could you please post your column headers from the exported file (from SecureSafe). The current importer expects the following column headers: Title,Username,Password,URL,Comment

Kind regards,
Daniel

@djsmith85 djsmith85 changed the title Import Fail SecureSafe: Import Fail Aug 29, 2024
@Diiieeegooo
Copy link
Author

Hi @Diiieeegooo and thank you for your report.

Could you please post your column headers from the exported file (from SecureSafe). The current importer expects the following column headers: Title,Username,Password,URL,Comment

Kind regards, Daniel

The Headline is like this but in germann

@djsmith85
Copy link
Contributor

Hi @Diiieeegooo and thank you for your report.
Could you please post your column headers from the exported file (from SecureSafe). The current importer expects the following column headers: Title,Username,Password,URL,Comment
Kind regards, Daniel

The Headline is like this but in germann

Thank you @Diiieeegooo for reporting back. To continue with you import please adjust the headers to the ones I mentioned.

Just for completeness would you mind posting the German headers here?

I have seen a similar report where the headers were exported in French.

Easiest fix will be to add a quick header replacement within the SecureSafe CSV importer. Marking this as good first issue.

@djsmith85 djsmith85 added browser Browser Extension cli CLI Application web good first issue labels Aug 29, 2024
@macronym
Copy link

macronym commented Sep 2, 2024

Hey @djsmith85 I am new to open-source contributions but I think I may be able to solve this issue. Could I be assigned?

@djsmith85
Copy link
Contributor

Hi @macronym ,

Thank you for offering your help with this. Github issues, don't get assigned, but feel free to open a Pull Request with your changes and link to this issue.

Please have a look at our Contribution Guidelines, these will help you with getting your environment set up and offer further information in regards to contributing.

Kind regards,
Daniel

@djsmith85
Copy link
Contributor

Here are the French columns headers provided by another user:
"Titre","Nom d'utilisateur","Mot de passe","URL","Commentaire"

@djsmith85
Copy link
Contributor

@macronym
Copy link

macronym commented Sep 2, 2024

Thank you for the tips! I claim this issue and will begin working on it. I believe I have gotten the development environment working.

macronym added a commit to macronym/clients that referenced this issue Sep 4, 2024
@macronym macronym linked a pull request Sep 4, 2024 that will close this issue
@djsmith85 djsmith85 linked a pull request Sep 4, 2024 that will close this issue
@tuhriel
Copy link

tuhriel commented Sep 24, 2024

I had the same issue recently and just found this issue while I planned to create a new one. The PR will probably solve the issue, but what I saw in my securesafe export is that the URL data contains additional slashes / probably to escape the slashes within the URLs:

"/Gmail/","/[email protected]/","/test","/https:////gmail.com/","comment/";

As far as I could see, the PR doesn't take that into account

I created a small python script to map the securesafe data to the default bitwarden csv-format. The csv module of python allows to set an escapechar which I defined as /. This cleaned the input completely and I just had to map it to the correct fields for the bitwarden format.

Not sure if the papaparse module of typescript has a similar function, that might help here

@macronym
Copy link

macronym commented Sep 24, 2024

@tuhriel I did consider that in my solution, Bitwarden seems to parse it correctly when there are four slashes in the URL, because in the app it shows only 2.

@ThexRazr ThexRazr linked a pull request Oct 4, 2024 that will close this issue
@djsmith85 djsmith85 linked a pull request Oct 4, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
browser Browser Extension bug cli CLI Application desktop Desktop Application good first issue web
Projects
None yet
4 participants