gnosis / cow-dex-solver

Other
56 stars 31 forks source link

Stricter limit price checks for 0x swaps #24

Closed MartinquaXD closed 2 years ago

MartinquaXD commented 2 years ago

The cowdexag solver on prod is producing a few alerts lately. The simulation shows that the solutions do not respect the users limit prices. At some point Valentin found that 0x buy orders did not respect the queried buy amount so we made limit price checks stricter on for all solvers in the services repo. I suspect the same issue could be causing the problems for the cowdexag solver so I lifted the code change from here and adapted it a bit.