[Bug]: bech32 should not be included in modules by default

[islishude]

Is there an existing issue for this?

• I have searched the existing issues

What happened?

cosmos sdk can provide custom address codec, but there are too many bech32 verification in the modules.

// ValidateBasic - validate transaction input
func (in Input) ValidateBasic() error {
	if _, err := sdk.AccAddressFromBech32(in.Address); err != nil {
		return sdkerrors.ErrInvalidAddress.Wrapf("invalid input address: %s", err)
	}

// ValidateBasic does basic validation on group info.
func (g GroupInfo) ValidateBasic() error {
	if g.Id == 0 {
		return errorsmod.Wrap(errors.ErrEmpty, "group's GroupId")
	}

	_, err := sdk.AccAddressFromBech32(g.Admin)
	if err != nil {
		return errorsmod.Wrap(err, "admin")
	}

Cosmos SDK Version

0.50.x

How to reproduce?

No response

[julienrbrt]

Yeah, that's true for v0.50 and lower. On main (future 0.52) this shouldn't be the case any more (ref: #7448). The address codec should be used everywhere.

[islishude]

Thanks for the heads up.

Where I can get the release plan?

I get the #20522 but it looks like the 0.51 will use the main branch

[julienrbrt]

Sorry, 0.51 vs 0.52 can be used interchangeably. The release be tagged as v0.52, but we call it sometime v0.51 by habit.