-
Notifications
You must be signed in to change notification settings - Fork 426
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove selfdestructs #25
Comments
Sure thing! 👍 |
What do you propose we do in this scenario? |
@nventuro I'd either remove the contract altogether as you mention, or change the |
This has now been fixed in |
Closed via #28 |
Until we support initialization of logic contracts in zOS, self-destructable contracts are open to being accidentally killed (see https://github.com/OpenZeppelin/openzeppelin-zos/blob/master/contracts/drafts/BreakInvariantBounty.sol#L68 for an example).
We should remove all selfdestruct calls from the oz-zos codebase.
Thanks @cwhinfrey for the report!
The text was updated successfully, but these errors were encountered: