Skip to content

Fix keystore KDF validation error argument names#5156

Open
yintaisha wants to merge 1 commit into
ethers-io:mainfrom
yintaisha:main
Open

Fix keystore KDF validation error argument names#5156
yintaisha wants to merge 1 commit into
ethers-io:mainfrom
yintaisha:main

Conversation

@yintaisha

Copy link
Copy Markdown

Fix two typo-level issues in JSON keystore KDF validation errors:

  • report invalid scrypt dklen with kdf.dklen instead of kdf.dflen
  • report invalid PBKDF2 prf with kdf.prf instead of kdf.pdf

This does not change successful keystore encryption or decryption behavior. It only makes the INVALID_ARGUMENT metadata and error message match the actual invalid fields.

Signed-off-by: yintaisha <yintaishan@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant