fix: avoid broadcasting tx through query (#15044)

(cherry picked from commit c884e27)
cosmos · Feb 16, 2023 · dd19da3 · dd19da3
1 parent 2e80d3b
commit dd19da3
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/baseapp/abci.go b/baseapp/abci.go
@@ -507,6 +507,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 {