LedgerCosmos.getAddressPubKeySECP256K1 sends back wrong HRP length recommendation to user

[odeke-em]

Please take a look at this length check

ledger-cosmos-go/user_app.go

Lines 294 to 295 in e9d4c19

	if len(hrp) > 83 {
	return nil, "", errors.New("hrp len should be <10")

It clearly guards that length of the HRP should be less than 83, but the error message says hrp len should be <10

/cc @elias-orijtech @julienrbrt

[jleni]

The error message is incorrect.

HRP must be between 1 and 83 bytes

https://github.com/bitcoin/bips/blob/master/bip-0173.mediawiki#bech32

This is also the limit in ledger apps.

https://github.com/cosmos/ledger-cosmos/blob/30910a6c0995e2d82ef1041c629440263799cafd/app/src/crypto.h#L29

The error message needs to be updated

[jleni]

fix in dev. it will be released soon