search

Synthetixio / synthetix

Synthetix Solidity smart contracts
https://synthetix.io/
MIT License
1.21k stars 600 forks source link

Deprecate selfDestructBeneficiary from SelfDestructible #581

Open hav-noms opened 4 years ago

hav-noms commented 4 years ago

The only Synthetix contract that holds ETH is EtherColalteral. So none of them need selfDestructBeneficiary. EtherColalteral does not implement SelfDestructible to give peace of mind to ETH depositors that the owner wont just SelfDestruct the contract and the selfDestructBeneficiary taking all their ETH.

dev1644 commented 4 years ago

Does this mean the removal of SelfDestructible contract?

pratokko commented 1 year ago

how did you solve the issue ? removing SelfDestructable instances from all the contracts for good experiences like @dev1644 mentioned