fix(client): write unbonding period advisory to stderr instead of std…

…out (#1921) * fix(client): write unbonding period advisory to stderr instead of stdout * Add changelog for unbonding period advisory --------- Co-authored-by: Philip Offtermatt <[email protected]>
cosmos · May 27, 2024 · 3132f64 · 3132f64
1 parent 16ee6a2
commit 3132f64
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.changelog/unreleased/bug-fixes/1921-write-stderr.md b/.changelog/unreleased/bug-fixes/1921-write-stderr.md
@@ -0,0 +1,2 @@
+- Write unbonding period advisory to stderr instead of stdout
+  ([\#1921](https://github.com/cosmos/interchain-security/pull/1921))
diff --git a/x/ccv/provider/client/proposal_handler.go b/x/ccv/provider/client/proposal_handler.go
@@ -363,7 +363,8 @@ func CheckPropUnbondingPeriod(clientCtx client.Context, propUnbondingPeriod time
 	providerUnbondingTime := res.Params.UnbondingTime
 
 	if providerUnbondingTime < propUnbondingPeriod {
-		fmt.Printf(
+		fmt.Fprintf(
+			os.Stderr,
 			`consumer unbonding period is advised to be smaller than provider unbonding period, but is longer.
 This is not a security risk, but will effectively lengthen the unbonding period on the provider.
 consumer unbonding: %s