Check for nil recipient before trying to access nested fields

gravitational · Jan 9, 2025 · 7b318dd · 7b318dd
1 parent 7e79e3e
commit 7b318dd
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/integrations/access/msteams/bot.go b/integrations/access/msteams/bot.go
@@ -188,11 +188,11 @@ func (b *Bot) FetchRecipient(ctx context.Context, recipient string) (*common.Rec
 	b.mu.RLock()
 	d, ok := b.fullRecipients[recipient]
 	b.mu.RUnlock()
-	rd, err := getRecipientData(&d)
-	if err != nil {
-		return nil, trace.Wrap(err)
-	}
 	if ok {
+		rd, err := getRecipientData(&d)
+		if err != nil {
+			return nil, trace.Wrap(err)
+		}
 		log.DebugContext(ctx, "Found recipient in cache",
 			slog.Group("recipient",
 				"id", d.ID,