chore: Optionally enable quote verification #2310
Labels
E:3.2 Verified Quotes
See https://github.com/cowprotocol/pm/issues/15 for details
Comments
MartinquaXD
added
the
E:3.2 Verified Quotes
This was referenced Jan 22, 2024
MartinquaXD
added a commit
that referenced
this issue
Jan 25, 2024
# Description Mostly moves code around and introduces trait for trade verification which will later be used for unit tests. Also adds the `verified` flag to quotes but at this point this flag is not yet used for anything. ## How to test Originally part of #2271 and changes were manually tested there. CI Fixes: #2310
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Background
Currently the frontend has to tell us that the backend should verify a quote by passing
priceQuality: "verified"in the/quoterequest.To easily enforce in the future that all quotes have been verified we need to make sure the backend is able to decide on its own which quotes to verify.
Details
We should start by verifying all quotes where
fromis not0and theprice_qualityis notfast.This should only happen conditionally based on a CLI flag in case we encounter any issues and have to quickly revert back to the status quo.
The text was updated successfully, but these errors were encountered: