fix: prevents sending very small balances when collecting fees from p…

…ools
White-Whale-Defi-Platform · Feb 6, 2024 · 916e651 · 916e651
1 parent a91b39b
commit 916e651
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/contracts/liquidity_hub/pool-network/terraswap_pair/Cargo.toml b/contracts/liquidity_hub/pool-network/terraswap_pair/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "terraswap-pair"
-version = "1.3.4"
+version = "1.3.5"
 authors = [
   "Terraform Labs, PTE.",
   "DELIGHT LABS",

diff --git a/contracts/liquidity_hub/pool-network/terraswap_pair/src/commands.rs b/contracts/liquidity_hub/pool-network/terraswap_pair/src/commands.rs
@@ -586,6 +586,8 @@ pub fn update_config(
     Ok(Response::new().add_attribute("action", "update_config"))
 }
 
+const MINIMUM_COLLECTABLE_BALANCE: Uint128 = Uint128::new(1_000u128);
+
 /// Collects all protocol fees accrued by the pool
 pub fn collect_protocol_fees(deps: DepsMut<TerraQuery>) -> Result<Response, ContractError> {
     let config = CONFIG.load(deps.storage)?;
@@ -610,7 +612,7 @@ pub fn collect_protocol_fees(deps: DepsMut<TerraQuery>) -> Result<Response, Cont
     let mut messages: Vec<CosmosMsg> = Vec::new();
     for protocol_fee in protocol_fees {
         // prevents trying to send 0 coins, which errors
-        if protocol_fee.amount != Uint128::zero() {
+        if protocol_fee.amount > MINIMUM_COLLECTABLE_BALANCE {
             messages.push(protocol_fee.into_msg(&deps.querier, config.fee_collector_addr.clone())?);
         }
     }