Commit f95b3140 authored by Maurelian's avatar Maurelian

feat(ctb): Remove ability to receive eth

We don't expect to need this, and it adds complexity to the message digest which
we'd prefer to avoid.
parent e096a62e
......@@ -96,18 +96,8 @@ contract DelayedVetoable is ISemver {
return _delay;
}
// TODO(maurelian): Remove this? The contract currently cannot handle forwarding ETH and I'm
// not sure the complexity is warranted.
// If we do allow it:
// 1. the callHash will need to include the value
// 2. forwarding will need to be done by passing the callHash, rather than the unhashed data
/// @notice Used when no data is passed to the contract.
receive() external payable {
_handleCall();
}
/// @notice Used for all calls that pass data to the contract.
fallback() external payable {
fallback() external {
_handleCall();
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment