From 429b7d154a3fb6f0a1ef82be79bf73f1b467748b Mon Sep 17 00:00:00 2001 From: "mergify[bot]" <37929162+mergify[bot]@users.noreply.github.com> Date: Tue, 25 Jan 2022 10:06:37 +0100 Subject: [PATCH] fix: return 404 on non-existing tx (backport #10992) (#11014) * fix: return 404 on non-existing tx (#10992) (cherry picked from commit 158128953c45f95fb4fd7ed737005b2bd9c9f1eb) # Conflicts: # CHANGELOG.md * fix cl Co-authored-by: Aleksandr Bezobchuk Co-authored-by: Aleksandr Bezobchuk --- CHANGELOG.md | 4 ++++ x/auth/tx/service.go | 2 ++ 2 files changed, 6 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 2b43ecfc01dd..d03919740051 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -38,6 +38,10 @@ Ref: https://keepachangelog.com/en/1.0.0/ ## [Unreleased] +### Bug Fixes + +* (grpc) [\#10985](https://github.com/cosmos/cosmos-sdk/pull/10992) The `/cosmos/tx/v1beta1/txs/{hash}` endpoint returns a 404 when a tx does not exist. + ## [v0.45.0](https://github.com/cosmos/cosmos-sdk/releases/tag/v0.45.0) - 2022-01-18 ### State Machine Breaking diff --git a/x/auth/tx/service.go b/x/auth/tx/service.go index b9633990d7ee..d95690cf108b 100644 --- a/x/auth/tx/service.go +++ b/x/auth/tx/service.go @@ -115,6 +115,8 @@ func (s txServer) GetTx(ctx context.Context, req *txtypes.GetTxRequest) (*txtype return nil, status.Error(codes.InvalidArgument, "tx hash cannot be empty") } + // TODO We should also check the proof flag in gRPC header. + // https://github.com/cosmos/cosmos-sdk/issues/7036. result, err := QueryTx(s.clientCtx, req.Hash) if err != nil { if strings.Contains(err.Error(), "not found") {