MetaMask / MetaMask/metamask-extension

Enforce checksummed addresses in send screen

Open
#4,869 1 comment 0 reactions 0 assignees View on GitHub
type-discussion type-enhancement
Dominant language
TypeScript
Stars
13.2k
Forks
5.6k
Avg merge
2d 5h
Merged PRs (30d)
451

Description

People lose money by hand typing addresses, so we can reduce those losses by using address checksums.

This means we wouldn't be compatible with all addresses.

How do we want to deal with that? A few options:

- Show a warning when sending to an un-checksummed address.
- Refuse to send to un-checksummed addresses.
- Allow user to force accept an un-checksummed address.
- Refuse to send to un-checksummed addresses specifically when typed in by the keyboard.

Contributor guide

Open the contributing guide

Research direction

Start with the send screen's address-entry and validation flow. Review the four handling options in the issue and determine which policy the project should adopt; done means the selected behavior is clearly defined and enforced for un-checksummed addresses.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
blockchain, frontend, security
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.