bounties icon indicating copy to clipboard operation
bounties copied to clipboard

[Ledger] Fix edit validator bug for removing bls key

Open gupadhyaya opened this issue 3 years ago • 16 comments

Description

Currently, there is a bug in the ledger firmware (https://github.com/harmony-one/ledger-app-one) that prevents validators from signing an edit validator transaction that tries to remove the bls key. All other edit validator options work correctly, only the remove-bls-key option fails with following error: Couldn't sign staking transaction, error: Error code 0x6802. This error may be due to invalid input bytes length sent to the ledger.

Context

  • Edit validator documentation: https://docs.harmony.one/home/network/validators/managing-a-validator/changing-validator-information
  • The error can also be reproduced using go-sdk (https://github.com/harmony-one/go-sdk), where you will use ./hmy -e for ledger
  • The error is reproducible for both nano s/x
  • The ledger nano s cost will be reimbursed

Acceptance Criteria

  • [ ] Fix the bug and make a PR to https://github.com/harmony-one/ledger-app-one repo

Reward

$1000 equivalent in ONE tokens

gupadhyaya avatar Apr 10 '21 03:04 gupadhyaya

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 6864.9452 ONE (998.47 USD @ $0.15/ONE) attached to it.

gitcoinbot avatar Apr 12 '21 19:04 gitcoinbot

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 8 months, 1 week ago. Please review their action plans below:

1) daychan has started work.

Check for the bugs. 2) yaclaudes1 has started work.

Will give it a try on a fix. The ledger hardware wallet ordered in regards to this project just arrived.

Learn more on the Gitcoin Issue Details page.

gitcoinbot avatar Apr 13 '21 10:04 gitcoinbot

Has the bug been fixed yet?

TheAlgorhythmicGuy avatar Apr 27 '21 08:04 TheAlgorhythmicGuy

I think I've fixed this issue as noted earlier, just testing it out now :)

mul1sh avatar May 03 '21 11:05 mul1sh

@mul1sh do you have the fix yet? thanks.

gupadhyaya avatar May 07 '21 03:05 gupadhyaya

@gupadhyaya Yes, but let me push tomorrow once I have tested in a nano X device which I'm purchasing today. So far I have done all my testing on a nano S, but I want to see if the fix works in the nano X and then I push 🙂

mul1sh avatar May 07 '21 06:05 mul1sh

It's still crashing for some reason in the nano X, it's what i'm fixing now

mul1sh avatar May 10 '21 14:05 mul1sh

got it. thanks @mul1sh please update here once you have the fix ready.

gupadhyaya avatar May 10 '21 15:05 gupadhyaya

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


The funding of 6864.9452 ONE (720.96 USD @ $0.11/ONE) attached to this issue has been cancelled by the bounty submitter

gitcoinbot avatar May 22 '21 04:05 gitcoinbot

@mul1sh any update on this? thanks.

gupadhyaya avatar Jun 03 '21 22:06 gupadhyaya

@gupadhyaya I thought the issue had been cancelled, so I stopped working on it. If it's still open, I'll PR by this coming Tuesday 🙂

mul1sh avatar Jun 04 '21 11:06 mul1sh

@gupadhyaya I thought the issue had been cancelled, so I stopped working on it. If it's still open, I'll PR by this coming Tuesday 🙂

it is still open @mul1sh https://gitcoin.co/issue/harmony-one/bounties/22/100025513, can you submit your work?

gupadhyaya avatar Jun 17 '21 14:06 gupadhyaya

@gupadhyaya Sure will push by this coming Monday, thanks for clarifying 🙂

mul1sh avatar Jun 18 '21 14:06 mul1sh

@mul1sh any update? if there is a blocker, I can take a look as well. thanks.

gupadhyaya avatar Jul 12 '21 19:07 gupadhyaya

Is this still open season? @gupadhyaya

matthewwoop avatar Jul 23 '21 03:07 matthewwoop

@mul1sh any update? if there is a blocker, I can take a look as well. thanks.

Sorry @gupadhyaya I was a bit caught up the last 2 months, but this week I have some free cycles and will solve this issue and PR.

mul1sh avatar Aug 09 '21 11:08 mul1sh