-
Notifications
You must be signed in to change notification settings - Fork 41
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
NodeJs: sweet improvements 🍓 #2161
Conversation
c05a3b7
to
59b60bd
Compare
Shouldn't we just overwrite it? I mean it's just an alias. |
could maybe be used to trick someone to enter the stronghold password ... I would be conservative here and not do that. I'ld rather have some explicit command that says |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe change the name of the PR now lol
WalletBuilder
Description of change
This PR:
adds a newWalletError
variant:AliasMismatch { provided: String, expected: String }
;returns that error in case a provided alias conflicts with the one stored in the wallet db;UPDATE: after a call we decided to remove this alias mismatch detection from this PR,
because we don't think there's a real problem with the builder overwriting the alias
stored in the db since this is non-critical data. I kept the other changes unrelated to
the wallet alias, which are:
wallet.spec.ts
testsLinks to any relevant issues
Fixes #2016
How the change has been tested
Integration tests.