entropyxyz / cli

CLI utility to ease interactions with an Entropy network.
GNU Affero General Public License v3.0
5 stars 0 forks source link

The CLI created an invalid address. #132

Closed johnnymatthews closed 4 months ago

johnnymatthews commented 4 months ago

This issue is based on a discussion in the Community repo: https://github.com/entropyxyz/community/discussions/54#discussioncomment-9839115

It looks like the CLI was able to generate an address that isn't a valid Substrate address. Based on the screenshot provided in the discussion, the CLI generated 5Equjw221oPGnrNjd3NLEZVYZCiKwtupU8ta1HaubkSsugrr as an address. When attempting to send funds to that address in the Substrate Portal, the portal complained that the address wasn't valid. Throwing the address into ss58.org also suggested that the address is invalid.

frankiebee commented 4 months ago

Oh no! This is no good currently we use polkadot js api's to generate are address's and secrets

frankiebee commented 4 months ago

@rh0delta notes: example function for validating an address

a possible temporary fix would be to check the address created by polkadot and to warn the user to create a new key?

this really does require some research. My gut reaction is to tell the user to generate a new seed after initial validation of the address. We might want to look into. Soft deriving a new address from the public key and seeing if that solves this issue without the creation of a new private key.

frankiebee commented 4 months ago

@johnnymatthews how was this address grabbed? the address by you posted by the user. When reading the screen shot in the thread: https://github.com/entropyxyz/community/discussions/54#discussioncomment-9839115 found the third to last carchter looked like a q but what you/them copied has a g.l when i swap that g for a q it's a valid address. did they possibly hand type it out?

ive highlighted a lower case g and the qthat was miss typed in this screen shot of theirs

Screenshot 2024-06-24 at 07 50 26
rh0delta commented 4 months ago

Looks like this issue is based off of an incorrect copy and paste from the user in question, going to close this for now

CC: @frankiebee @johnnymatthews