Reject orders using too much gas

[fleupold]

Description

We are seeing inefficiency with some ERC1271 orders on Gnosis that are using a large amount of gas reducing settlement throughput for other traders. This PR limits the amount of gas an order can consume (including signature validation and hooks)

Changes

• Introduce a new error type too much gas
• Return this error if the total gas estimate (including hooks) exceeds a certain threshold

Considerations:

• Should we also block native ETH flow orders below a certain validation limit?
• Should we already warn during quoting when the order is likely going to exceed the gas limit?

Both of these would make the PR more involved while not directly addressing the somewhat pressing issue we have on Gnosis Chain.

How to test

Test for hooks. I'm also working on a test for ERC1271 orders, however it's a bit more involved because we need a fake verifier contract that uses a lot of gas

[squadgazzz]

LG. Regarding the considerations, maybe we should analyze how often the error occurs once the change is released.

[MartinquaXD]

@mfw78 this introduces a new error type the watchtower needs to know about to not get stuck indexing these orders, right?

[mfw78]

cowprotocol/cow-sdk#202