search

ToucanProtocol / dynamic-fee-pools

Fee calculator contracts for Toucan Protocol pools
2 stars 3 forks source link

`feeSetup` needs to be guarded #13

Closed 0xmichalis closed 11 months ago

0xmichalis commented 11 months ago

Currently feeSetup can be called by anyone. I suggest we update the contract to be Ownable then only the contract owner should be allowed to call feeSetup. A multisig co-owned by Neutral and Toucan could then be set up in mainnet as the contract owner.

https://github.com/neutral-protocol/dynamic-fee-pools/blob/fec047ceef8e4a610f919ac350b894bae672ea4b/src/FeeCalculator.sol#L36

PawelTroka commented 11 months ago

done via #22