ibc: use outbound_ics20_transfers_enabled chain parameter (#4101)

this uses the existing outbound_ics20_transfers_enabled to gate ics20 withdrawals
penumbra-zone · Mar 26, 2024 · 83f5311 · 83f5311
1 parent 2058857
commit 83f5311
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/crates/core/app/src/action_handler/actions.rs b/crates/core/app/src/action_handler/actions.rs
@@ -84,7 +84,18 @@ impl AppActionHandler for Action {
                     .check_stateful(state)
                     .await
             }
-            Action::Ics20Withdrawal(action) => action.check_historical(state).await,
+            Action::Ics20Withdrawal(action) => {
+                // SAFETY: this is safe to check in parallel because IBC enablement cannot
+                // change during transaction execution.
+                if !state
+                    .get_ibc_params()
+                    .await?
+                    .outbound_ics20_transfers_enabled
+                {
+                    anyhow::bail!("transaction an ICS20 withdrawal, but outbound ICS20 withdrawals are not enabled");
+                }
+                action.check_historical(state).await
+            }
             Action::CommunityPoolSpend(action) => action.check_historical(state).await,
             Action::CommunityPoolOutput(action) => action.check_historical(state).await,
             Action::CommunityPoolDeposit(action) => action.check_historical(state).await,