zeruniverse / Password-Manager

An online keepass-like tool to manage password. client-side AES encryption!
Other
170 stars 44 forks source link

changing user password error #135

Closed BenjaminHae closed 7 years ago

BenjaminHae commented 7 years ago

When I try to change my login password for the manager I get the error "ERROR: empty key detected!". But the process still continues, everything ends fine.(I tried my own installation and the test-platform on sourceforge)

BenjaminHae commented 7 years ago

I think the problem is "fname": encryptchar(accountarray[x]["fname"],newsecretkey) when fname is empty. The line can probably just be deleted, but as I have no idea how the file things work I'm not sure.

zeruniverse commented 7 years ago

I'll take a look.

On Sun, Mar 12, 2017 at 9:16 AM Benjamin Häublein notifications@github.com wrote:

I think the problem is "fname": encryptchar(accountarray[x]["fname"],newsecretkey) and it can probably be just deleted. But as I have no idea how the file things work I'm not sure.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/zeruniverse/Password-Manager/issues/135#issuecomment-285955164, or mute the thread https://github.com/notifications/unsubscribe-auth/AEbvNFplckk_nsVslqD8T3j3B_1cF4Z8ks5rlBpdgaJpZM4MalV9 .

zeruniverse commented 7 years ago

Yes, it's caused by empty fname. Quite strange that I didn't encounter into the problem yet...