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

BIP-44 based HD wallet in mycelo #1903

Open
hiromaily opened this issue May 24, 2022 · 0 comments
Open

BIP-44 based HD wallet in mycelo #1903

hiromaily opened this issue May 24, 2022 · 0 comments
Labels

Comments

@hiromaily
Copy link

hiromaily commented May 24, 2022

Description

For now, HD wallet derivation path is not based on BIP-44 in mycelo. This difference makes key synchronization among other libraries difficult. Proper path must be like m / purpose' / coin_type' / account' / change / address_index. This is specification.
e.g. "m/44'/60'/0'/0/0" is correct, but current"m/1/0" is not. mycelo code

For example, developer account keys are generated after running geth nodes with specific mnemonic through mycelo. Then I want to deploy contracts onto celo geth nodes. However it's difficult to generate same address with mnemonic using other libraries like @truffle/hdwallet-provider in truffle-config.json

Current Behavior

HD wallet derivation path is not based on BIP-44 in mycelo

Context

This is BIP-44 specification.

Proposed Solution

To follow bip-44, m / purpose' / coin_type' / account' / change / address_index,
only coin_type need to be fixed which value must be used in mycelo. But I'd like to pass this value when running mycelo command.

@37ng 37ng assigned 37ng and unassigned 37ng May 24, 2022
@carterqw2 carterqw2 added triage Issue needs triaging component: mycelo mycelo-related issues and removed blockchain labels Feb 9, 2023
@carterqw2 carterqw2 added good first issue and removed triage Issue needs triaging labels Feb 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants
@hiromaily @carterqw2 @37ng and others