diff --git a/CHANGELOG.md b/CHANGELOG.md index b7ebfa96423c..1c974e07cd2b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -46,6 +46,7 @@ Ref: https://keepachangelog.com/en/1.0.0/ * (testing) [#5](https://github.com/01builders/cosmos-sdk/pull/5) Add `MakeTestEncodingConfigWithOpts` to pass `testutil.CodecOptions` when creating `TestEncodingConfig`. * (x/bank) [#6](https://github.com/01builders/cosmos-sdk/pull/6) Change denom index bank migration to noOp. * (x/auth) [#7](https://github.com/01builders/cosmos-sdk/pull/7) Support legacy global AccountNumber. +* (crypto/keyring) [#8](https://github.com/01builders/cosmos-sdk/pull/8) Fix error wrapping in pkg crypto/keyring. Removed `cockroachdb/errors` in favour of `errorsmod.Wrap`. ## [v0.50.11](https://github.com/cosmos/cosmos-sdk/releases/tag/v0.50.11) - 2024-12-16 diff --git a/crypto/keyring/keyring.go b/crypto/keyring/keyring.go index 6598aa88738f..46dde35c5659 100644 --- a/crypto/keyring/keyring.go +++ b/crypto/keyring/keyring.go @@ -3,6 +3,7 @@ package keyring import ( "bufio" "encoding/hex" + "errors" "fmt" "io" "os" @@ -11,7 +12,6 @@ import ( "strings" "github.com/99designs/keyring" - "github.com/cockroachdb/errors" "github.com/cosmos/go-bip39" "golang.org/x/crypto/bcrypt" @@ -418,7 +418,7 @@ func (ks keystore) SaveLedgerKey(uid string, algo SignatureAlgo, hrp string, coi priv, _, err := ledger.NewPrivKeySecp256k1(*hdPath, hrp) if err != nil { - return nil, errors.CombineErrors(ErrLedgerGenerateKey, err) + return nil, errorsmod.Wrap(err, ErrLedgerGenerateKey.Error()) } return ks.writeLedgerKey(uid, priv.PubKey(), hdPath) @@ -813,7 +813,7 @@ func (ks keystore) writeRecord(k *Record) error { serializedRecord, err := ks.cdc.Marshal(k) if err != nil { - return errors.CombineErrors(ErrUnableToSerialize, err) + return errorsmod.Wrap(err, ErrUnableToSerialize.Error()) } item := keyring.Item{ @@ -964,7 +964,7 @@ func (ks keystore) migrate(key string) (*Record, error) { serializedRecord, err := ks.cdc.Marshal(k) if err != nil { - return nil, errors.CombineErrors(ErrUnableToSerialize, err) + return nil, errorsmod.Wrap(err, ErrUnableToSerialize.Error()) } item = keyring.Item{