chore: make fee aggregation permissionless

contracts/liquidity_hub/fee_collector/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "fee_collector"
-version = "1.1.2"
+version = "1.1.3"
 authors = ["Kerber0x <[email protected]>"]
 edition.workspace = true
 description = "Contract to collect the fees accrued by the pools and vaults in the liquidity hub"

contracts/liquidity_hub/fee_collector/src/commands.rs

@@ -170,17 +170,22 @@ pub fn update_config(
 /// Aggregates the fees collected into the given asset_info.
 pub fn aggregate_fees(
     mut deps: DepsMut,
-    info: MessageInfo,
     env: Env,
-    ask_asset_info: AssetInfo,
+    mut ask_asset_info: AssetInfo,
     aggregate_fees_for: FeesFor,
 ) -> Result<Response, ContractError> {
     let config: Config = CONFIG.load(deps.storage)?;
 
-    // only the owner or the contract itself can aggregate the fees
-    if info.sender != config.owner && info.sender != env.contract.address {
-        return Err(ContractError::Unauthorized {});
-    }
+    // query fee collector
+    let fee_distributor_config: white_whale::fee_distributor::Config =
+        deps.querier.query(&QueryRequest::Wasm(WasmQuery::Smart {
+            contract_addr: config.fee_distributor.to_string(),
+            msg: to_binary(&white_whale::fee_distributor::QueryMsg::Config {})?,
+        }))?;
+
+    // This was done in order to make this function permissionless so anyone can trigger the fee aggregation
+    // The signature for `ExecuteMsg::AggregateFees` was kept the same to avoid migrating multiple contracts
+    ask_asset_info = fee_distributor_config.distribution_asset;
 
     let mut aggregate_fees_messages: Vec<CosmosMsg> = Vec::new();
 

contracts/liquidity_hub/fee_collector/src/contract.rs

@@ -128,7 +128,7 @@ pub fn execute(
         ExecuteMsg::AggregateFees {
             asset_info,
             aggregate_fees_for,
-        } => commands::aggregate_fees(deps, info, env, asset_info, aggregate_fees_for),
+        } => commands::aggregate_fees(deps, env, asset_info, aggregate_fees_for),
         ExecuteMsg::ForwardFees {
             epoch,
             forward_fees_as,