connector/sync: fix NPE when syncing channels

Before this change syncChannels would NPE if inboxData.Conversations was nil due to being omitted because it's empty. This change ensures that the Conversations map is not nil before copying to it. Signed-off-by: Sumner Evans <[email protected]>
mautrix · Jan 14, 2025 · c36c257 · c36c257
1 parent 40c3f56
commit c36c257
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/pkg/connector/client_sync.go b/pkg/connector/client_sync.go
@@ -44,7 +44,13 @@ func (tc *TwitterClient) syncChannels(ctx context.Context, initialInboxState *re
 			break
 		}
 
+		if inboxData.Conversations == nil {
+			inboxData.Conversations = map[string]types.Conversation{}
+		}
 		maps.Copy(inboxData.Conversations, nextInboxTimelineResponse.InboxTimeline.Conversations)
+		if inboxData.Users == nil {
+			inboxData.Users = map[string]types.User{}
+		}
 		maps.Copy(inboxData.Users, nextInboxTimelineResponse.InboxTimeline.Users)
 		inboxData.Entries = append(inboxData.Entries, nextInboxTimelineResponse.InboxTimeline.Entries...)