Hi,
For the receive()
issue mentioned by others
- OpenZeppelin upgradeable contracts affected by Istanbul hardfork
- Msg.sender.transfer runs out of gas on a payable upgradeable proxy contract
If I change the code here
from
receive() external payable virtual {
_fallback();
}
to
receive() external payable virtual {
}
Looks like for my case, the issue will be fixed. Wondering if it's an OK way to work around this just for our use case and wondering if there's any potential pitfalls here?
We are deploying a UUPS upgradable contracts which we wish we can directly send ETH to it. The contract itself does not have too much functionalities
thanks!