Velvet-Capital / Velvet-v4

V4 (thena+venus) on top of v3
Other
0 stars 0 forks source link

Enhance Oracle Failure Mitigation Strategies #32

Closed langnavina97 closed 4 days ago

langnavina97 commented 1 month ago

Background

Our current system relies heavily on price oracles for critical operations such as token swaps and liquidity provision. While we have an upgradeable oracle in place, there might be a need to implement additional safeguards to protect against potential oracle failures or manipulations.

Current Risks

Proposed Enhancements

  1. Fallback Mechanism for Fee Reinvestment
    • Allow asset managers to reinvest fees by specifying a minimum amount out, bypassing the oracle
    • This enables continued operation during oracle downtime
aj07 commented 1 month ago

Bit very edge case.. ( but we had some issue in v2 where chain link stopped supporting the price feed for some tokens). There user had option to do multitoken withdrawal.

Here, as an alternate we can have "Allow asset managers to reinvest fees by specifying a minimum amount out" and i was thinking we can think of alternate oracle this time...if we all agree we can take the bnb oracle

langnavina97 commented 1 month ago

Using the function updateFeed we could update the aggregator and deploy our own contract (aggregator) to interact with any oracle.. @Havoc19 what do you think?

Havoc19 commented 4 weeks ago

Using the function updateFeed we could update the aggregator and deploy our own contract (aggregator) to interact with any oracle.. @Havoc19 what do you think?

Here for this case, we should test with one more oracle or else when we are actually going to change it may not work

aj07 commented 4 weeks ago

https://docs.pyth.network/home

langnavina97 commented 2 weeks ago

Can we close this issue after testing the integration of another oracle https://github.com/Velvet-Capital/Velvet-v4/tree/price-oracle-test? @aj07 @Havoc19