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

Failed to import wallet #1799

Closed
udikantz opened this issue Mar 3, 2019 · 8 comments

Comments

2 participants
@udikantz
Copy link

commented Mar 3, 2019

Hello,
I am using latest node release self built on centos 7,

When executing wallet_import command I am getting a strange error massage,
how ever after getting the error massage I am executing the wallet_list command and I see the "failed imported wallet" listed under the wallet_list

here is the error massage:


nano_node: /home/leetz0rd/latest_node/nano_build/nano/node/wallet.cpp:580: bool nano::wallet_store::import(const nano::transaction&, nano::wallet_store&): Assertion `valid_password (transaction_a)' failed.
Aborted
@SergiySW

This comment has been minimized.

Copy link
Collaborator

commented Mar 3, 2019

Wallet you import to shouldn't have password (you can change to empty password before import I guess, after import change again)

@udikantz

This comment has been minimized.

Copy link
Author

commented Mar 3, 2019

I am using the command as described in the docs:

./nano_node --wallet_import --file=/home/crypto/nano/data/backup/<old wallet>.json --wallet=<new wallet> --password=<old wallet password> --data_path=/home/crypto/nano/data18/

@SergiySW

This comment has been minimized.

Copy link
Collaborator

commented Mar 3, 2019

If your new wallet has password, then you cannot import old wallet json

@udikantz

This comment has been minimized.

Copy link
Author

commented Mar 3, 2019

I see, the new wallet indeed had password prior to import.
Strangely after getting the error, I am seeing the imported wallet as if it was imported correctly.

@SergiySW

This comment has been minimized.

Copy link
Collaborator

commented Mar 3, 2019

It seems during import temporary wallet with random id was created. It should have all json content including keys & seed encrypted with json password. Because you are using Debug build, process was cancelled with assert & temporary wallet remains
Should be more clear with #1800

@udikantz

This comment has been minimized.

Copy link
Author

commented Mar 3, 2019

Thank you for the answers,
I did not know i was using debug build, are there instruction to build the node for production?

@SergiySW

This comment has been minimized.

Copy link
Collaborator

commented Mar 3, 2019

Add to cmake options
-DCMAKE_BUILD_TYPE=Release

@SergiySW SergiySW added this to CP1 (2019-03-13) in V19 Mar 4, 2019

@SergiySW SergiySW self-assigned this Mar 4, 2019

@SergiySW SergiySW added the bug label Mar 4, 2019

@SergiySW SergiySW added this to the V19.0 milestone Mar 4, 2019

@SergiySW

This comment has been minimized.

Copy link
Collaborator

commented Mar 6, 2019

Bug should be fixed in v19/master with #1800. Returning explaining message if new wallet is encrypted with different password

@SergiySW SergiySW closed this Mar 6, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.