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

gaiacli can't delete ledger keys #2921

Closed
4 tasks
alessio opened this issue Nov 28, 2018 · 2 comments
Closed
4 tasks

gaiacli can't delete ledger keys #2921

alessio opened this issue Nov 28, 2018 · 2 comments
Assignees
Labels
C:Keys Keybase, KMS and HSMs T:Bug

Comments

@alessio
Copy link
Contributor

alessio commented Nov 28, 2018

Steps to Reproduce

alessio@bangalter:~/.../cosmos/cosmos-sdk$ gaiacli --home=./gaiacli keys add --ledger ledgerKey
NAME:	TYPE:	ADDRESS:						PUBKEY:
ledgerKey	ledger	cosmos19h5kgyernhlxlurqpztk5h45aqt7h4hhscv9at	cosmospub1addwnpepq0le7np5nfgjxgqxyapgnsza9kz86w49lzcuetescrpq3ey25zsks2cszgm
alessio@bangalter:~/.../cosmos/cosmos-sdk$ gaiacli --home=./gaiacli keys list
NAME:	TYPE:	ADDRESS:						PUBKEY:
ledgerKey	ledger	cosmos19h5kgyernhlxlurqpztk5h45aqt7h4hhscv9at	cosmospub1addwnpepq0le7np5nfgjxgqxyapgnsza9kz86w49lzcuetescrpq3ey25zsks2cszgm
alessio@bangalter:~/.../cosmos/cosmos-sdk$ gaiacli --home=./gaiacli keys delete ledgerKey
DANGER - enter password to permanently delete key:
ERROR: password must be at least 8 characters
alessio@bangalter:~/.../cosmos/cosmos-sdk$ gaiacli --home=./gaiacli keys list
NAME:	TYPE:	ADDRESS:						PUBKEY:
ledgerKey	ledger	cosmos19h5kgyernhlxlurqpztk5h45aqt7h4hhscv9at	cosmospub1addwnpepq0le7np5nfgjxgqxyapgnsza9kz86w49lzcuetescrpq3ey25zsks2cszgm
alessio@bangalter:~/.../cosmos/cosmos-sdk$ gaiacli --home=./gaiacli keys delete ledgerKey
DANGER - enter password to permanently delete key:
Password deleted forever (uh oh!)
alessio@bangalter:~/.../cosmos/cosmos-sdk$ gaiacli --home=./gaiacli keys list
NAME:	TYPE:	ADDRESS:						PUBKEY:
ledgerKey	ledger	cosmos19h5kgyernhlxlurqpztk5h45aqt7h4hhscv9at	cosmospub1addwnpepq0le7np5nfgjxgqxyapgnsza9kz86w49lzcuetescrpq3ey25zsks2cszgm

For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@alessio alessio added T:Bug C:Keys Keybase, KMS and HSMs labels Nov 28, 2018
@alessio alessio changed the title gaiacli can't to delete ledger keys gaiacli can't delete ledger keys Nov 28, 2018
@alexanderbez
Copy link
Contributor

Thanks @alessio!

/cc @cwgoes I doubt this is intended behavior?

@cwgoes
Copy link
Contributor

cwgoes commented Nov 28, 2018

It is definitely not intended behaviour! Also, we should clarify the UX to make it obvious that the user is just deleting a public key (not the private key from the Ledger - that cannot be done with gaiacli).

@alessio alessio self-assigned this Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:Keys Keybase, KMS and HSMs T:Bug
Projects
None yet
Development

No branches or pull requests

3 participants