Skip to content

Commit

Permalink
fix: update enforce sub-authenticator to be greater than 1 error mess…
Browse files Browse the repository at this point in the history
…age (#8676)

* Fix composite authenticator error msg

* update changelog
  • Loading branch information
iboss-ptk authored Sep 6, 2024
1 parent 6a32298 commit 17054cf
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
* [#8628](https://github.com/osmosis-labs/osmosis/pull/8628) chore: add tagged cometbft version: v0.38.11-v26-osmo-1
* [#8649](https://github.com/osmosis-labs/osmosis/pull/8649) chore: update to tagged submodules
* [#8663](https://github.com/osmosis-labs/osmosis/pull/8663) fix: protorev throws a nil pointer
* [#8676](https://github.com/osmosis-labs/osmosis/pull/8676) fix: update enforce sub-authenticator to be greater than 1 error message

### Config

Expand Down
2 changes: 1 addition & 1 deletion x/smart-account/authenticator/composite.go
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ func onSubAuthenticatorsAdded(ctx sdk.Context, account sdk.AccAddress, data []by
}

if len(initDatas) <= 1 {
return errorsmod.Wrap(sdkerrors.ErrInvalidRequest, "no sub-authenticators provided")
return errorsmod.Wrapf(sdkerrors.ErrInvalidRequest, "at least 2 sub-authenticators must be provided, but got %d", len(initDatas))
}

baseId := authenticatorId
Expand Down

0 comments on commit 17054cf

Please sign in to comment.