mleku/wasmd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add logging for query err before redacting (#1605)

Browse Source 
* Add logging for query err before redacting

* Fix comments
This commit is contained in:
pinosu
2023-09-06 11:42:16 +02:00
committed by GitHub
parent 086c052c18
commit f1835a13e4
2 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
x/wasm/keeper/query_plugins.go
View File

@@ -71,6 +71,7 @@ func (q QueryHandler) Query(request wasmvmtypes.QueryRequest, gasLimit uint64) (
}
// Issue #759 - we don't return error string for worries of non-determinism
moduleLogger(q.Ctx).Debug("Redacting submessage error", "cause", err)
return nil, redactError(err)
}

3
x/wasm/keeper/query_plugins_test.go
View File

@@ -10,6 +10,7 @@ import (
wasmvmtypes "github.com/CosmWasm/wasmvm/types"
dbm "github.com/cometbft/cometbft-db"
"github.com/cometbft/cometbft/libs/log"
"github.com/cometbft/cometbft/libs/rand"
tmproto "github.com/cometbft/cometbft/proto/tendermint/types"
"github.com/cosmos/gogoproto/proto"
@@ -688,7 +689,7 @@ func TestQueryErrors(t *testing.T) {
mock := keeper.WasmVMQueryHandlerFn(func(ctx sdk.Context, caller sdk.AccAddress, request wasmvmtypes.QueryRequest) ([]byte, error) {
return nil, spec.src
})
ctx := sdk.Context{}.WithGasMeter(sdk.NewInfiniteGasMeter()).WithMultiStore(store.NewCommitMultiStore(dbm.NewMemDB()))
ctx := sdk.Context{}.WithGasMeter(sdk.NewInfiniteGasMeter()).WithMultiStore(store.NewCommitMultiStore(dbm.NewMemDB())).WithLogger(log.TestingLogger())
q := keeper.NewQueryHandler(ctx, mock, sdk.AccAddress{}, keeper.NewDefaultWasmGasRegister())
_, gotErr := q.Query(wasmvmtypes.QueryRequest{}, 1)
assert.Equal(t, spec.expErr, gotErr)