Another batch of low-priority issues is fixed in AMM.vy and Controller.vy (pointed out by Chainsecurity and Mixbytes, as well as internally by @Macket). Biggest fixes are:
- health_calculator() was not working properly (an external contract is currently used for it). Only affects convenience of integrators;
- It is was still possible for governance to reset user’s LTV, although not by design, if combined with AMM manipulation. In this fix, even governance vote cannot change LTV unexpectedly for a user (only after user’s interaction with the AMM).
The fixes are contained from commit
Etherscan doesn’t support blueprints as of now, so verification can only be done by comparing raw bytecode with code of blueprints as produced by deployment script.
How to verify
Since Etherscan doesn’t support blueprints, one has to use the deployment script to verify:
$ ape run scripts/ape-deploy-amm-controller-Aug17.py verify --network ethereum:mainnet:alchemy Blueprints match the ones deployed on chain