Open petrovska-petro opened 6 days ago
It is not intended for the injector to hold gas tokens. I guess then could just remove this function. This is just kind of the baseline of what I have used as a admin owned, fully controlled, max flexibillity helper.
Then, indeed makes sense to remove withdrawGasToken
from the contract
Severity: if ether is sent without defining the
receive()
method in the smart contract, it will throw an exceptionContext: the method
withdrawGasToken
suggests that gas balance will be in the contract, but no proper way to received actual gas in the contractRecommendation: see documentation and add at least a
receive()
method if intention is to receive gas tokens into the injector.Review stage
Balancer Maxis:
Onchainification Labs: