addresses.json should produce checksum addresses

makerdao / dss-deploy-scripts

GNU Affero General Public License v3.0

49 stars 59 forks source link

addresses.json should produce checksum addresses #79

Closed EdNoepel closed 4 years ago

EdNoepel commented 4 years ago

Checksum addresses have some security benefits. It'd be nice if contract and token addresses in addresses.json used them.

DONALDSULLENDER commented 4 years ago

im so confused.

EdNoepel commented 4 years ago

This was resolved. After deployment, an addresses.json file is produced enumerating the address for each contract deployed. It used to produce this file with addresses in all lowercase. Now it produces the file using checksum addresses.