From 4969ae9358cdf44d3d32f69005683ecc4cb63989 Mon Sep 17 00:00:00 2001
From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com>
Date: Tue, 14 Jun 2022 15:09:40 +0200
Subject: [PATCH] feat: added check for wildcard * to allow all message types
 (#1512) (#1537)

* added check for wildcard * to allow all message types

* update docs

* nit

(cherry picked from commit 08d91ada9c3a1cb3394c047b2364481404ec6742)

Co-authored-by: Charly <charly@interchain.berlin>
---
 docs/apps/interchain-accounts/parameters.md   |  8 ++++
 .../host/keeper/relay_test.go                 | 39 +++++++++++++++++++
 .../27-interchain-accounts/host/types/keys.go |  5 +++
 3 files changed, 52 insertions(+)

diff --git a/docs/apps/interchain-accounts/parameters.md b/docs/apps/interchain-accounts/parameters.md
index f4ddcbc1160..8f985b9293e 100644
--- a/docs/apps/interchain-accounts/parameters.md
+++ b/docs/apps/interchain-accounts/parameters.md
@@ -47,4 +47,12 @@ For example, a Cosmos SDK based chain that elects to provide hosted Interchain A
     "host_enabled": true,
     "allow_messages": ["/cosmos.staking.v1beta1.MsgDelegate", "/cosmos.gov.v1beta1.MsgVote"]
 }
+```
+There is also a special wildcard `"*"` message type which allows any type of message to be executed by the interchain account. This must be the only message in the `allow_messages` array.
+
+```
+"params": {
+    "host_enabled": true,
+    "allow_messages": ["*"]
+}
 ```
\ No newline at end of file
diff --git a/modules/apps/27-interchain-accounts/host/keeper/relay_test.go b/modules/apps/27-interchain-accounts/host/keeper/relay_test.go
index 4de981cbc14..76fe2c96a4f 100644
--- a/modules/apps/27-interchain-accounts/host/keeper/relay_test.go
+++ b/modules/apps/27-interchain-accounts/host/keeper/relay_test.go
@@ -29,6 +29,45 @@ func (suite *KeeperTestSuite) TestOnRecvPacket() {
 		malleate func()
 		expPass  bool
 	}{
+		{
+			"interchain account successfully executes an arbitrary message type using the * (allow all message types) param",
+			func() {
+				interchainAccountAddr, found := suite.chainB.GetSimApp().ICAHostKeeper.GetInterchainAccountAddress(suite.chainB.GetContext(), ibctesting.FirstConnectionID, path.EndpointA.ChannelConfig.PortID)
+				suite.Require().True(found)
+
+				// Populate the gov keeper in advance with an active proposal
+				testProposal := &govtypes.TextProposal{
+					Title:       "IBC Gov Proposal",
+					Description: "tokens for all!",
+				}
+
+				proposal, err := govtypes.NewProposal(testProposal, govtypes.DefaultStartingProposalID, time.Now(), time.Now().Add(time.Hour))
+				suite.Require().NoError(err)
+
+				suite.chainB.GetSimApp().GovKeeper.SetProposal(suite.chainB.GetContext(), proposal)
+				suite.chainB.GetSimApp().GovKeeper.ActivateVotingPeriod(suite.chainB.GetContext(), proposal)
+
+				msg := &govtypes.MsgVote{
+					ProposalId: govtypes.DefaultStartingProposalID,
+					Voter:      interchainAccountAddr,
+					Option:     govtypes.OptionYes,
+				}
+
+				data, err := icatypes.SerializeCosmosTx(suite.chainA.GetSimApp().AppCodec(), []sdk.Msg{msg})
+				suite.Require().NoError(err)
+
+				icaPacketData := icatypes.InterchainAccountPacketData{
+					Type: icatypes.EXECUTE_TX,
+					Data: data,
+				}
+
+				packetData = icaPacketData.GetBytes()
+
+				params := types.NewParams(true, []string{"*"})
+				suite.chainB.GetSimApp().ICAHostKeeper.SetParams(suite.chainB.GetContext(), params)
+			},
+			true,
+		},
 		{
 			"interchain account successfully executes banktypes.MsgSend",
 			func() {
diff --git a/modules/apps/27-interchain-accounts/host/types/keys.go b/modules/apps/27-interchain-accounts/host/types/keys.go
index 416599ea83f..d0c0b0da643 100644
--- a/modules/apps/27-interchain-accounts/host/types/keys.go
+++ b/modules/apps/27-interchain-accounts/host/types/keys.go
@@ -14,6 +14,11 @@ const (
 
 // ContainsMsgType returns true if the sdk.Msg TypeURL is present in allowMsgs, otherwise false
 func ContainsMsgType(allowMsgs []string, msg sdk.Msg) bool {
+	// check that wildcard * option for allowing all message types is the only string in the array, if so, return true
+	if len(allowMsgs) == 1 && allowMsgs[0] == "*" {
+		return true
+	}
+
 	for _, v := range allowMsgs {
 		if v == sdk.MsgTypeURL(msg) {
 			return true