Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

sc-keystore: Improve weird error message #10584

Merged
merged 1 commit into from
Jan 5, 2022

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Jan 4, 2022

The keystore would print "Invalid password" when a key was stored using an incorrect public key.
This pr improves the error message to communicate better to the user on what is wrong.

The keystore would print "Invalid password" when a key was stored using an incorrect public key.
This pr improves the error message to communicate better to the user on what is wrong.
@bkchr bkchr added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Jan 4, 2022
@andresilva
Copy link
Contributor

bot merge

@paritytech-processbot
Copy link

Bot will approve on the behalf of @andresilva, since they are a team lead, in an attempt to reach the minimum approval count

@paritytech-processbot paritytech-processbot bot merged commit 2ae6242 into master Jan 5, 2022
@paritytech-processbot paritytech-processbot bot deleted the bkchr-improve-keystore-message branch January 5, 2022 12:09
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
The keystore would print "Invalid password" when a key was stored using an incorrect public key.
This pr improves the error message to communicate better to the user on what is wrong.
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
The keystore would print "Invalid password" when a key was stored using an incorrect public key.
This pr improves the error message to communicate better to the user on what is wrong.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants