From b152c03f98704a07a6e2241acb03c0fe3a5c8aa1 Mon Sep 17 00:00:00 2001
From: Dmitry Kolupaev <dmitry.klpv@gmail.com>
Date: Wed, 6 Sep 2023 16:00:35 +0400
Subject: [PATCH] fix: remove addr validation for provider fee pool addr param
 (#1262)

* fix: remove validation for provider chain address since we cannot validate it properly on consumer

* add changelog entry
---
 CHANGELOG.md                        | 1 +
 x/ccv/consumer/types/params_test.go | 4 ----
 x/ccv/types/params.go               | 4 ++--
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 353d7226f1..792e83a7a9 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -26,6 +26,7 @@ Add an entry to the unreleased consumer section whenever merging a PR to main th
 * (fix!) [#1146](https://github.com/cosmos/interchain-security/pull/1146) Proper deletion of pending packets.
 * (feat!) [#1037](https://github.com/cosmos/interchain-security/pull/1037) Optimize pending packets storage on consumer, with migration. 
 * (feat) [#1164](https://github.com/cosmos/interchain-security/pull/1164) Introduce the gRPC query `/interchain_security/ccv/consumer/provider-info` and CLI command `interchain-security-cd q ccvconsumer provider-info` to retrieve provider info from the consumer chain.
+* (fix!) [#1262](https://github.com/cosmos/interchain-security/pull/1262) Remove incorrect address validation on `ProviderFeePoolAddrStr` param 
 
 ## v3.1.0
 
diff --git a/x/ccv/consumer/types/params_test.go b/x/ccv/consumer/types/params_test.go
index 1690fc9f81..ff5d0f325f 100644
--- a/x/ccv/consumer/types/params_test.go
+++ b/x/ccv/consumer/types/params_test.go
@@ -29,10 +29,6 @@ func TestValidateParams(t *testing.T) {
 			"custom invalid params, dist transmission channel",
 			ccvtypes.NewParams(true, 5, "badchannel/", "", 5, 1005, "0.5", 1000, 24*21*time.Hour, "0.05", []string{"untrn"}, []string{"uatom"}), false,
 		},
-		{
-			"custom invalid params, provider fee pool addr string",
-			ccvtypes.NewParams(true, 5, "", "imabadaddress", 5, 1005, "0.5", 1000, 24*21*time.Hour, "0.05", []string{"untrn"}, []string{"uatom"}), false,
-		},
 		{
 			"custom invalid params, ccv timeout",
 			ccvtypes.NewParams(true, 5, "", "", -5, 1005, "0.5", 1000, 24*21*time.Hour, "0.05", []string{"untrn"}, []string{"uatom"}), false,
diff --git a/x/ccv/types/params.go b/x/ccv/types/params.go
index 2b553e8366..6a903c0d8f 100644
--- a/x/ccv/types/params.go
+++ b/x/ccv/types/params.go
@@ -179,8 +179,8 @@ func ValidateProviderFeePoolAddrStr(i interface{}) error {
 	if i == "" {
 		return nil
 	}
-	// Otherwise validate as usual for a bech32 address
-	return ValidateBech32(i)
+	// Cannot validate provider chain address on the consumer chain
+	return nil
 }
 
 func ValidateSoftOptOutThreshold(i interface{}) error {