Skip to content

Commit 3ab56f8

Browse files
authored
Merge pull request #396 from smacker/fix_event_type_log_field
fix: Use string instead of reflect.Type as log field
2 parents bfdc972 + c6b806f commit 3ab56f8

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

server/server.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ func NewServer(
5757
// HandleEvent processes the event calling the analyzers, and posting the results
5858
func (s *Server) HandleEvent(ctx context.Context, e lookout.Event) error {
5959
ctx, logger := ctxlog.WithLogFields(ctx, log.Fields{
60-
"event-type": reflect.TypeOf(e),
60+
"event-type": reflect.TypeOf(e).String(),
6161
"event-id": e.ID().String(),
6262
"repo": e.Revision().Head.InternalRepositoryURL,
6363
"head": e.Revision().Head.ReferenceName,

0 commit comments

Comments
 (0)