Add basic address check before sending

Lamden / wallet

Official Lamden wallet as a Google Chrome plugin. Also includes native atomic swap tools.

24 stars 19 forks source link

Add basic address check before sending #152

Closed Endogen closed 2 years ago

Endogen commented 3 years ago

Since at least one user already sent his mainnet TAU to an Ethereum wallet: Consider adding a basic check to validate the to address before sending:

Is the address exactly 64 chars long?
Is it a valid hex string?

duckfev3r commented 3 years ago

This 100% needs to be addressed..

Since it's possible (and desirable) to send funds to a smart contract, the above solution won't work. Perhaps the rules could be :

if the address starts with "con_" => trim whitespace from each end
if not, check it's a valid Lamden address (correct length & characters)

RIP 2000 $RSWP

JeffWScott commented 2 years ago

added in release 1.9.1