fix #2850

[goreleaser]
synapsecns · Jul 7, 2024 · 8048ad1 · 8048ad1
1 parent d787c29
commit 8048ad1
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/contrib/opbot/botmd/middleware.go b/contrib/opbot/botmd/middleware.go
@@ -23,7 +23,7 @@ func (b *Bot) tracingMiddleware() slacker.CommandMiddlewareHandler {
 		return func(cmdCtx *slacker.CommandContext) {
 			ctx, span := b.handler.Tracer().Start(cmdCtx.Context(), fmt.Sprintf("command.%s", cmdCtx.Definition().Command), trace.WithAttributes(
 				attribute.String("user_id", cmdCtx.Event().UserID),
-				attribute.String("channel_id", cmdCtx.Event().Channel.ID),
+				attribute.String("channel_id", retrieveChannelIfExists(cmdCtx.Event())),
 			))
 
 			cmdCtx.WithContext(ctx)
@@ -37,6 +37,15 @@ func (b *Bot) tracingMiddleware() slacker.CommandMiddlewareHandler {
 	}
 }
 
+const unknownChannel = "unknown"
+
+func retrieveChannelIfExists(event *slacker.MessageEvent) string {
+	if event != nil && event.Channel != nil {
+		return event.Channel.ID
+	}
+	return unknownChannel
+}
+
 // assumes method is only called once.
 type otelRecorder struct {
 	attemptsCounter       metric.Int64UpDownCounter