api: make protoFormat a bit safer to use

Give it a better name, as it has little to do with formatting.
Make it return bytes, which is what we need.
Don't let it silently ignore errors, which could cause issues.
Use the Marshal protobuf func, as Format is only for debugging.

api/chain.go

@@ -706,7 +706,7 @@ func (a *API) chainTxHandler(_ *apirest.APIdata, ctx *httprouter.HTTPContext) er
 		return ErrVochainGetTxFailed.WithErr(err)
 	}
 	tx := &GenericTransactionWithInfo{
-		TxContent: []byte(protoFormat(stx.Tx)),
+		TxContent: protoTxAsJSON(stx.Tx),
 		Signature: stx.Signature,
 		TxInfo:    *ref,
 	}

api/helpers.go

@@ -57,17 +57,21 @@ func (a *API) sendTx(tx []byte) (*cometcoretypes.ResultBroadcastTx, error) {
 	return resp, nil
 }
 
-func protoFormat(tx []byte) string {
+func protoTxAsJSON(tx []byte) []byte {
 	ptx := models.Tx{}
 	if err := proto.Unmarshal(tx, &ptx); err != nil {
-		return ""
+		panic(err)
 	}
 	pj := protojson.MarshalOptions{
 		Multiline:       false,
 		Indent:          "",
 		EmitUnpopulated: true,
 	}
-	return pj.Format(&ptx)
+	asJSON, err := pj.Marshal(&ptx)
+	if err != nil {
+		panic(err)
+	}
+	return asJSON
 }
 
 // isTransactionType checks if the given transaction is of the given type.

api/helpers_test.go

@@ -155,7 +155,7 @@ func TestConvertKeysToCamel(t *testing.T) {
 	}
 }
 
-func TestProtoFormat(t *testing.T) {
+func TestProtoTxAsJSON(t *testing.T) {
 	wantJSON := strings.TrimSpace(`
 {
 	"setProcess": {
@@ -176,7 +176,7 @@ func TestProtoFormat(t *testing.T) {
 	qt.Assert(t, err, qt.IsNil)
 
 	var dst bytes.Buffer
-	err = json.Indent(&dst, []byte(protoFormat(asProto)), "", "\t")
+	err = json.Indent(&dst, protoTxAsJSON(asProto), "", "\t")
 	gotJSON := dst.String()
 
 	qt.Assert(t, gotJSON, qt.Equals, wantJSON)