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

When importing accounts with a csv file, passwords don't get imported #373

Closed
anogues opened this issue Jan 18, 2017 · 20 comments
Closed

When importing accounts with a csv file, passwords don't get imported #373

anogues opened this issue Jan 18, 2017 · 20 comments
Assignees
Milestone

Comments

@anogues
Copy link

anogues commented Jan 18, 2017

They appear blank.

Also in the db the column is empty

@nuxsmin nuxsmin self-assigned this Jan 18, 2017
@nuxsmin nuxsmin added this to the 2.0 milestone Jan 18, 2017
@nuxsmin
Copy link
Owner

nuxsmin commented Jan 18, 2017

Did you set the master password option?. If so, it's only needed when importing XML files (doc needed), so if you set it, it was used so the process tried to decrypt every password with it.

@anogues
Copy link
Author

anogues commented Jan 18, 2017

it may well be yes. I thin i tried to import the csv file with the master password (no import password set).

I will retry tomorrow without setting the master password and i let you know!

Thanks

@nuxsmin
Copy link
Owner

nuxsmin commented Jan 18, 2017

Ok, the documentation is already being written so it would be available when the stable version were released.

@anogues
Copy link
Author

anogues commented Jan 19, 2017

Uhm something not working.

I am importing the following csv:

p11;server;unix;url;user;password;notes

no passwords set, neither master nor import

Importación finalizada
Revise el registro de eventos para más detalles

but then i go to check the account:

image

It appears empty, i think it is because not setting the master password it hasn't cyphered the account when inserting it to the db

mysql> select account_pass from accounts where account_name = 'p11';
+--------------+
| account_pass |
+--------------+
| password |
+--------------+
1 row in set (0.00 sec)

But what i want is to import the csv file, and then store the encrypted passwords in the database, like when you create manually an account.

So i tried adding the master password in the import stage with the following

p12;server;unix;url;user;password;notes

but then:

Importación finalizada
Revise el registro de eventos para más detalles

Same behaviour in the UI and the field is empty in the db:
mysql> select account_pass from accounts where account_name = 'p12';
+--------------+
| account_pass |
+--------------+
| |
+--------------+
1 row in set (0.00 sec)

So how can i supply clear passwords in a csv file and get them encrypted on the db when these are bulk loaded?

nuxsmin added a commit that referenced this issue Jan 19, 2017
@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

I'm blind....I didn't notice that the password was encrypted if it was previously decrypted...

@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

I'm solving an issue with the CSV delimiter...I'll let you know when solved

nuxsmin added a commit that referenced this issue Jan 19, 2017
@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

Hi, it has been solved in the latest commit (devel branch). I've added the field enclosure character ("), so fields would be more safely parsed. Does it break your CSV?

@anogues
Copy link
Author

anogues commented Jan 19, 2017

No, we can add the quotes, no problem.

I'll try!

@anogues
Copy link
Author

anogues commented Jan 19, 2017

Con las quotes no funciona, vuelve a salir el password en blanco!

@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

Es raro...
seleccion_089

Tiene algún carácter especial?

@anogues
Copy link
Author

anogues commented Jan 19, 2017

No, estoy probando esto

@anogues
Copy link
Author

anogues commented Jan 19, 2017

"p12";"server";"unix";"url";"user";"password";"notes"

@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

Igual...
syspass csv -dev-res - gedit_090

@anogues
Copy link
Author

anogues commented Jan 19, 2017

Pruebo de nuevo a ver si la he liado..

@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

Actualizaste al último commit? https://github.com/nuxsmin/sysPass/commits/devel

@anogues
Copy link
Author

anogues commented Jan 19, 2017

Si, le he dado a bajar el ultimo zip de devel

@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

Es más, lo último que he añadido es la ayuda sobre los campos

@anogues
Copy link
Author

anogues commented Jan 19, 2017

Ya está ya está, le habia puesto el master password...

Culpa mía

@nuxsmin
Copy link
Owner

nuxsmin commented Jan 19, 2017

Ammm...perfecto!!

@anogues
Copy link
Author

anogues commented Jan 19, 2017

Todo cargado ya, y funcionando, gracias!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants