Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

add optional chainId in checksum #2

Closed
gasolin opened this issue Nov 14, 2021 · 1 comment
Closed

add optional chainId in checksum #2

gasolin opened this issue Nov 14, 2021 · 1 comment

Comments

@gasolin
Copy link

gasolin commented Nov 14, 2021

evaluate if worth to support EIP-1191 (add chain id to mixed-case checksum address)
ethereum/EIPs#1121

which is already supported in several utilities
https://github.com/ethereumjs/ethereumjs-util/blob/master/src/account.ts#L134

basically it helps different EVM compatible chain users to aware to not send their token to other chain's address (via encode chain id in address)

generateAddress(addr, chainId = 1)
checkAddress(addr, chainId = 1)

@gasolin
Copy link
Author

gasolin commented Nov 14, 2021

more discussion in PaulRBerg/ethsum#5 , close the issue

@gasolin gasolin closed this as completed Nov 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant