From c884e276f2c41477011dd160e23ff448d8c5db29 Mon Sep 17 00:00:00 2001
From: Facundo Medica <14063057+facundomedica@users.noreply.github.com>
Date: Thu, 16 Feb 2023 12:13:23 -0300
Subject: [PATCH] fix: avoid broadcasting tx through query (#15044)

---
 baseapp/abci.go | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/baseapp/abci.go b/baseapp/abci.go
index 25df8c918b..62b3fad846 100644
--- a/baseapp/abci.go
+++ b/baseapp/abci.go
@@ -520,6 +520,10 @@ func (app *BaseApp) Query(req abci.RequestQuery) (res abci.ResponseQuery) {
 	telemetry.IncrCounter(1, "query", req.Path)
 	defer telemetry.MeasureSince(time.Now(), req.Path)
 
+	if req.Path == "/cosmos.tx.v1beta1.Service/BroadcastTx" {
+		return sdkerrors.QueryResult(sdkerrors.Wrap(sdkerrors.ErrInvalidRequest, "can't route a broadcast tx message"), app.trace)
+	}
+
 	// handle gRPC routes first rather than calling splitPath because '/' characters
 	// are used as part of gRPC paths
 	if grpcHandler := app.grpcQueryRouter.Route(req.Path); grpcHandler != nil {