missing error message when calling GetTxsEvent

[pacmessica]

Summary of Bug

When calling grpc method GetTxsEvent, error logs are always empty. You only include ABCILogs in the types.TxResponse, so if log is an error string then it's never part of the response. Might need to add RawLog field?

from here

func txResultToTxResponse(respTx *abci.ResponseDeliverTx) *sdk.TxResponse {
	logs, _ := sdk.ParseABCILogs(respTx.Log)
	return &sdk.TxResponse{
		Code:      respTx.Code,
		Codespace: respTx.Codespace,
		GasUsed:   respTx.GasUsed,
		GasWanted: respTx.GasWanted,
		Info:      respTx.Info,
		Logs:      logs,
	}
}

using rpc /tx_search?query="tx.height=5201364" I can see the error log as string (whereas this error is completely missing in grpc response):

                "tx_result": {
                    "code": 5,
                    "data": null,
                    "log": "failed to execute message; message index: 0: failed to delegate; 4044574uatom is smaller than 4047074uatom: insufficient funds",
                    "info": "",
                    "gas_wanted": "250000",
                    "gas_used": "78452",
                    "events": [],
                    "codespace": "sdk"
                },

Version

v0.41.3

Steps to Reproduce

github.com/cosmos/cosmos-sdk/types/tx

Create a tx.NewServiceClient(cli) where cli is a *grpc.ClientConn to a cosmoshub-4 node. Call GetTxsEvent with service client for a height with a known error, something like:

		grpcRes, err := c.txServiceClient.GetTxsEvent(nctx, &tx.GetTxsEventRequest{
			Events:     []string{"tx.height=5201364"},
		}

I expect to see the error message in types.TxResponse

		for i, trans := range grpcRes.Txs {
			resp := grpcRes.TxResponses[i]  // missing RawLog, no sign of error string
		}

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[amaury1093]

Might need to add RawLog field?

Sounds right to me. Would you like to create a PR?

Edit: The other fields on sdk.TxResponse should also be populated, e.g. #8680