From 51cdc5cead47e8d34f8dd4af18a7139c9ff41059 Mon Sep 17 00:00:00 2001 From: PaddyMc Date: Wed, 4 Sep 2024 18:10:45 +0100 Subject: [PATCH] fix: smart account signing checktx error (#8665) * fix: smart account signing checktx error * chore: add changelog entry --- CHANGELOG.md | 1 + x/smart-account/authenticator/authentication_request.go | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index dbb99fe9899..5c3d6fdec70 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -69,6 +69,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 * [#8494](https://github.com/osmosis-labs/osmosis/pull/8494) Add additional events in x/lockup, x/superfluid, x/concentratedliquidity * [#8543](https://github.com/osmosis-labs/osmosis/pull/8543) Add OTEL wiring and new configs in app.toml * [#8566](https://github.com/osmosis-labs/osmosis/pull/8566) Minor speedup to CalcExitCFMM shares +* [#8665](https://github.com/osmosis-labs/osmosis/pull/8665) fix: smart account signing checktx error ## v25.2.1 diff --git a/x/smart-account/authenticator/authentication_request.go b/x/smart-account/authenticator/authentication_request.go index 85e99cf3a54..c128280b9e7 100644 --- a/x/smart-account/authenticator/authentication_request.go +++ b/x/smart-account/authenticator/authentication_request.go @@ -245,8 +245,8 @@ func GenerateAuthenticationRequest( AuthenticatorParams: nil, } - // We do not generate the sign bytes if simulate is true - if simulate { + // We do not generate the sign bytes if simulate is true or isCheckTx is true + if simulate && ctx.IsCheckTx() { return authRequest, nil }