online cache

internal/msggateway/user_map.go

@@ -118,12 +118,12 @@ func (u *userMap) DeleteClients(userID string, clients []*Client) (isDeleteUser
 	tmp := result.Clients
 	result.Clients = result.Clients[:0]
 	for _, client := range tmp {
-		if _, ok := deleteAddr[client.ctx.GetRemoteAddr()]; ok {
+		if _, delCli := deleteAddr[client.ctx.GetRemoteAddr()]; delCli {
-			continue
-		}
 			offline = append(offline, int32(client.PlatformID))
+		} else {
 			result.Clients = append(result.Clients, client)
 		}
+	}
 	defer u.push(userID, result, offline)
 	if len(result.Clients) > 0 {
 		return false

pkg/rpccache/online.go

@@ -51,11 +51,8 @@ func NewOnlineCache(user rpcclient.UserRpcClient, group *GroupLocalCache, rdb re
 				log.ZError(ctx, "OnlineCache redis subscribe parseUserOnlineStatus", err, "payload", message.Payload, "channel", message.Channel)
 				continue
 			}
-			log.ZDebug(ctx, "OnlineCache setUserOnline", "userID", userID, "platformIDs", platformIDs, "payload", message.Payload)
+			storageCache := x.setUserOnline(userID, platformIDs)
-			x.setUserOnline(userID, platformIDs)
+			log.ZDebug(ctx, "OnlineCache setUserOnline", "userID", userID, "platformIDs", platformIDs, "payload", message.Payload, "storageCache", storageCache)
-			//if err := x.setUserOnline(ctx, userID, platformIDs); err != nil {
-			//	log.ZError(ctx, "redis subscribe setUserOnline", err, "payload", message.Payload, "channel", message.Channel)
-			//}
 		}
 	}()
 	return x
@@ -115,10 +112,10 @@ func (o *OnlineCache) GetGroupOnline(ctx context.Context, groupID string) ([]str
 			onlineUserIDs = append(onlineUserIDs, userID)
 		}
 	}
-	log.ZDebug(ctx, "OnlineCache GetGroupOnline", "groupID", groupID, "onlineUserIDs", onlineUserIDs)
+	log.ZDebug(ctx, "OnlineCache GetGroupOnline", "groupID", groupID, "onlineUserIDs", onlineUserIDs, "allUserID", userIDs)
 	return onlineUserIDs, nil
 }
 
-func (o *OnlineCache) setUserOnline(userID string, platformIDs []int32) {
+func (o *OnlineCache) setUserOnline(userID string, platformIDs []int32) bool {
-	o.local.SetHas(o.getUserOnlineKey(userID), platformIDs)
+	return o.local.SetHas(o.getUserOnlineKey(userID), platformIDs)
 }