code/push: way more debugging

Author: mfycheng

pkg/code/push/data.go

@@ -99,6 +99,8 @@ func sendMutableNotificationToOwner(
 		return err
 	}
 
+	log.WithField("tokens", pushTokenRecords).Info("Found push tokens")
+
 	seenPushTokens := make(map[string]struct{})
 	for _, pushTokenRecord := range pushTokenRecords {
 		// Dedup push tokens, since they may appear more than once per app install
@@ -112,6 +114,7 @@ func sendMutableNotificationToOwner(
 		var err error
 		switch pushTokenRecord.TokenType {
 		case push_data.TokenTypeFcmApns:
+			log.Info("Sending mutable push")
 			err = pusher.SendMutableAPNSPush(
 				ctx,
 				pushTokenRecord.PushToken,
@@ -122,6 +125,7 @@ func sendMutableNotificationToOwner(
 			)
 		case push_data.TokenTypeFcmAndroid:
 			// todo: anything special required for Android?
+			log.Info("Sending data push")
 			err = pusher.SendDataPush(
 				ctx,
 				pushTokenRecord.PushToken,

pkg/code/server/grpc/chat/v2/server.go

@@ -791,6 +791,8 @@ func (s *Server) SendMessage(ctx context.Context, req *chatpb.SendMessageRequest
 	}
 
 	s.onPersistChatMessage(log, chatId, chatMessage)
+
+	log.Info("Sending push notifications from SendMessage()")
 	s.sendPushNotifications(chatId, chatTitle, memberId, chatMessage)
 
 	return &chatpb.SendMessageResponse{
@@ -1423,11 +1425,20 @@ func (s *Server) sendPushNotifications(chatId chat.ChatId, chatTitle string, sen
 		return
 	}
 
+	log.WithField("members", members).Info("Found members for push")
+
 	var eg errgroup.Group
 	eg.SetLimit(min(32, len(members)))
 
 	for _, m := range members {
+		log := log.WithField("member", m.MemberId.String())
 		if m.MemberId == sender || m.IsMuted || m.IsUnsubscribed {
+			log.WithFields(logrus.Fields{
+				"isSender":       m.MemberId == sender,
+				"isMuted":        m.IsMuted,
+				"isUnsubscribed": m.IsUnsubscribed,
+			}).Info("skipping member")
+
 			continue
 		}