ChainlinkOEVWrapper + ChainlinkOEVMorphoWrapper

[imthatcarlos]

changes

• integration + unit tests for ChainlinkOEVWrapper and ChainlinkOEVMorphoWrapper
• mip-x37

todo

• update integration tests to loop over every token wrapper and execute assertions
• update script to only deploy one implementation
• proper liquidation setup in integration tests
• for core oev wrapperrs, use _addReserves on the collateral mtoken as the fee recipient (ie no need for feeRecipient)
• for morpho oev wrappers, use the equivalent core market as the feeRecipient on deployment
• update integration tests
• add integration tests against real 10/10 liquidations to get the fees that would've been collected if these oev wrappers were up
• confirm we call setFeed for all markets

[anajuliabit]

Would be great to mock liqudity on morpho so we always execute the test no matter the onchain state

[imthatcarlos]

i removed the try...catch and it seems the borrow goes through for all of them.

[anajuliabit]

Allowing the caller to pass the mToken looks dangeours to me because they can deploy a fake mToken and try to steal the contract money.

What are yours thoughts?

[imthatcarlos]

good catch, we need to whitelist those mTokens

[anajuliabit]

lets not use msg.sender here, add the owner as a parameter, this should be the temporal goverrnor

[imthatcarlos]

I transfer ownership in the proposal script, after whitelisting all the markets

[openzeppelin-code]

ChainlinkOEVWrapper + ChainlinkOEVMorphoWrapper

Generated at commit: 427b23414928078ad68a3745ec8c479ea9ca712b

🚨 Report Summary

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	4 4 0 12 46 66
Dependencies	Critical High Medium Low Note Total	0 0 0 0 0 0

---

For more details view the full report in OpenZeppelin Code Inspector