Pretty print Broker ResponseMessage

cytechmobile · Mar 20, 2024 · 8ffd90a · 8ffd90a
1 parent 30dfc33
commit 8ffd90a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/app/broker/broker.go b/app/broker/broker.go
@@ -106,6 +106,11 @@ type ResponseMessage struct {
 	ResultDetails []WorkflowDetails `json:"-"`
 }
 
+func (rm *ResponseMessage) String() string {
+	return fmt.Sprintf("ResponseMessage{Response:%+v, RunID:%+v, Result:%+v, ResultDetails:%+v}", rm.Response,
+		*rm.RunID, rm.Result, rm.ResultDetails)
+}
+
 type WorkflowDetails struct {
 	WorkflowID     string `json:"workflow_id"`
 	WorkflowName   string `json:"workflow_name"`

diff --git a/cmd/github-actions-adapter/serve/serve.go b/cmd/github-actions-adapter/serve/serve.go
@@ -2,7 +2,6 @@ package serve
 
 import (
 	"context"
-	"fmt"
 	"log/slog"
 	"radicle-github-actions-adapter/app"
 	"radicle-github-actions-adapter/app/broker"
@@ -62,7 +61,7 @@ func (gas *GitHubActionsServer) Serve(ctx context.Context) error {
 			ID: eventUUID,
 		},
 	}
-	gas.App.Logger.Debug("sending message", "message", fmt.Sprintf("%+v", jobResponse))
+	gas.App.Logger.Debug("sending message", "message", jobResponse)
 	err = gas.Broker.ServeResponse(ctx, jobResponse)
 	if err != nil {
 		gas.App.Logger.Error("could not send response message to broker", "error", err.Error())
@@ -109,7 +108,7 @@ func (gas *GitHubActionsServer) Serve(ctx context.Context) error {
 			_ = gas.commentOnPatch(ctx, brokerRequestMessage, commentMessage)
 		}
 	}
-	gas.App.Logger.Debug("sending message", "message", fmt.Sprintf("%+v", resultResponse))
+	gas.App.Logger.Debug("sending message", "message", resultResponse)
 	err = gas.Broker.ServeResponse(ctx, resultResponse)
 	if err != nil {
 		gas.App.Logger.Error("could not send response message to broker", "error", err.Error())