From ecb9599b25fde0e3569b46e667ca1990294f8665 Mon Sep 17 00:00:00 2001
From: liusuyi <1951119284@qq.com>
Date: Tue, 16 Dec 2025 10:26:21 +0800
Subject: [PATCH] 修复【在线用户强制退出正在使用的账号】bug

---
 ard-framework/src/main/java/com/ard/framework/security/handle/CustomAuthenticationSuccessHandler.java |    3 +
 ard-framework/src/main/java/com/ard/framework/web/service/SysLoginService.java                        |    2 
 ard-admin/src/main/java/com/ard/web/controller/monitor/SysUserOnlineController.java                   |   55 +++++++++++++--------------
 ard-framework/src/main/java/com/ard/framework/web/service/TokenService.java                           |    2 
 ard-common/src/main/java/com/ard/common/constant/Constants.java                                       |    5 --
 ard-common/src/main/java/com/ard/common/constant/CacheConstants.java                                  |    5 ++
 6 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/ard-admin/src/main/java/com/ard/web/controller/monitor/SysUserOnlineController.java b/ard-admin/src/main/java/com/ard/web/controller/monitor/SysUserOnlineController.java
index 1cc4a66..3f17016 100644
--- a/ard-admin/src/main/java/com/ard/web/controller/monitor/SysUserOnlineController.java
+++ b/ard-admin/src/main/java/com/ard/web/controller/monitor/SysUserOnlineController.java
@@ -4,6 +4,7 @@
 import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
+
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.security.access.prepost.PreAuthorize;
 import org.springframework.web.bind.annotation.DeleteMapping;
@@ -25,13 +26,12 @@
 
 /**
  * 在线用户监控
- * 
+ *
  * @author ard
  */
 @RestController
 @RequestMapping("/monitor/online")
-public class SysUserOnlineController extends BaseController
-{
+public class SysUserOnlineController extends BaseController {
     @Autowired
     private ISysUserOnlineService userOnlineService;
 
@@ -40,27 +40,18 @@
 
     @PreAuthorize("@ss.hasPermi('monitor:online:list')")
     @GetMapping("/list1")
-    public TableDataInfo list1(String ipaddr, String userName)
-    {
+    public TableDataInfo list1(String ipaddr, String userName) {
         Collection<String> keys = redisCache.keys(CacheConstants.LOGIN_TOKEN_KEY + "*");
         List<SysUserOnline> userOnlineList = new ArrayList<SysUserOnline>();
-        for (String key : keys)
-        {
+        for (String key : keys) {
             LoginUser user = redisCache.getCacheObject(key);
-            if (StringUtils.isNotEmpty(ipaddr) && StringUtils.isNotEmpty(userName))
-            {
+            if (StringUtils.isNotEmpty(ipaddr) && StringUtils.isNotEmpty(userName)) {
                 userOnlineList.add(userOnlineService.selectOnlineByInfo(ipaddr, userName, user));
-            }
-            else if (StringUtils.isNotEmpty(ipaddr))
-            {
+            } else if (StringUtils.isNotEmpty(ipaddr)) {
                 userOnlineList.add(userOnlineService.selectOnlineByIpaddr(ipaddr, user));
-            }
-            else if (StringUtils.isNotEmpty(userName) && StringUtils.isNotNull(user.getUser()))
-            {
+            } else if (StringUtils.isNotEmpty(userName) && StringUtils.isNotNull(user.getUser())) {
                 userOnlineList.add(userOnlineService.selectOnlineByUserName(userName, user));
-            }
-            else
-            {
+            } else {
                 userOnlineList.add(userOnlineService.loginUserToUserOnline(user));
             }
         }
@@ -68,10 +59,10 @@
         userOnlineList.removeAll(Collections.singleton(null));
         return getDataTable(userOnlineList);
     }
+
     @PreAuthorize("@ss.hasPermi('monitor:online:list')")
     @GetMapping("/list")
-    public TableDataInfo list(String ipaddr, String userName)
-    {
+    public TableDataInfo list(String ipaddr, String userName) {
         startPage();  // ❗ Ruoyi 分页
 
         Collection<String> keys = redisCache.keys(CacheConstants.LOGIN_TOKEN_KEY + "*");
@@ -80,8 +71,7 @@
         boolean searchIp = StringUtils.isNotEmpty(ipaddr);
         boolean searchName = StringUtils.isNotEmpty(userName);
 
-        for (String key : keys)
-        {
+        for (String key : keys) {
             LoginUser loginUser = redisCache.getCacheObject(key);
             if (loginUser == null || loginUser.getUser() == null)
                 continue;
@@ -93,15 +83,13 @@
             boolean match = true;
 
             // ⭐ IP 模糊匹配
-            if (searchIp)
-            {
+            if (searchIp) {
                 String userIp = loginUser.getIpaddr();
                 match = userIp != null && userIp.contains(ipaddr);
             }
 
             // ⭐ 用户名模糊匹配
-            if (match && searchName)
-            {
+            if (match && searchName) {
                 String nick = loginUser.getUsername();
                 match = nick != null && nick.contains(userName);
             }
@@ -120,9 +108,18 @@
     @PreAuthorize("@ss.hasPermi('monitor:online:forceLogout')")
     @Log(title = "在线用户", businessType = BusinessType.FORCE)
     @DeleteMapping("/{tokenId}")
-    public AjaxResult forceLogout(@PathVariable String tokenId)
-    {
-        redisCache.deleteObject(CacheConstants.LOGIN_TOKEN_KEY + tokenId);
+    public AjaxResult forceLogout(@PathVariable String tokenId) {
+        String tokenKey = CacheConstants.LOGIN_TOKEN_KEY + tokenId;
+
+        LoginUser loginUser = redisCache.getCacheObject(tokenKey);
+
+        if (loginUser != null && loginUser.getUser() != null) {
+            Long userId = loginUser.getUser().getUserId();
+            redisCache.deleteObject(CacheConstants.LOGIN_USERID_KEY + userId);
+        }
+
+        redisCache.deleteObject(tokenKey);
+
         return success();
     }
 }
diff --git a/ard-common/src/main/java/com/ard/common/constant/CacheConstants.java b/ard-common/src/main/java/com/ard/common/constant/CacheConstants.java
index a7adf51..5dd4433 100644
--- a/ard-common/src/main/java/com/ard/common/constant/CacheConstants.java
+++ b/ard-common/src/main/java/com/ard/common/constant/CacheConstants.java
@@ -13,6 +13,11 @@
     public static final String LOGIN_TOKEN_KEY = "login_tokens:";
 
     /**
+     * 登录用户编号 redis key
+     */
+    public static final String LOGIN_USERID_KEY = "login_userid:";
+
+    /**
      * 验证码 redis key
      */
     public static final String CAPTCHA_CODE_KEY = "captcha_codes:";
diff --git a/ard-common/src/main/java/com/ard/common/constant/Constants.java b/ard-common/src/main/java/com/ard/common/constant/Constants.java
index 152af4e..9e17143 100644
--- a/ard-common/src/main/java/com/ard/common/constant/Constants.java
+++ b/ard-common/src/main/java/com/ard/common/constant/Constants.java
@@ -171,8 +171,5 @@
     public static final String[] JOB_ERROR_STR = { "java.net.URL", "javax.naming.InitialContext", "org.yaml.snakeyaml",
             "org.springframework", "org.apache", "com.ard.common.utils.file", "com.ard.common.config", "com.ard.generator" };
 
-    /**
-     * 登录用户编号 redis key
-     */
-    public static final String LOGIN_USERID_KEY = "login_userid:";
+
 }
diff --git a/ard-framework/src/main/java/com/ard/framework/security/handle/CustomAuthenticationSuccessHandler.java b/ard-framework/src/main/java/com/ard/framework/security/handle/CustomAuthenticationSuccessHandler.java
index 5744902..5733a29 100644
--- a/ard-framework/src/main/java/com/ard/framework/security/handle/CustomAuthenticationSuccessHandler.java
+++ b/ard-framework/src/main/java/com/ard/framework/security/handle/CustomAuthenticationSuccessHandler.java
@@ -1,5 +1,6 @@
 package com.ard.framework.security.handle;
 
+import com.ard.common.constant.CacheConstants;
 import com.ard.common.constant.Constants;
 import com.ard.common.core.redis.RedisCache;
 import org.springframework.beans.factory.annotation.Autowired;
@@ -13,7 +14,7 @@
 
     public void onAuthenticationSuccess(Long userId) {
         // 检查 Redis 中是否已有该用户登录
-        String key = Constants.LOGIN_USERID_KEY + userId;
+        String key = CacheConstants.LOGIN_USERID_KEY + userId;
         if (redisCache.hasKey(key)) {
             throw new RuntimeException("用户已登录,请勿重复登录");
         }
diff --git a/ard-framework/src/main/java/com/ard/framework/web/service/SysLoginService.java b/ard-framework/src/main/java/com/ard/framework/web/service/SysLoginService.java
index 1c5aced..1d4773c 100644
--- a/ard-framework/src/main/java/com/ard/framework/web/service/SysLoginService.java
+++ b/ard-framework/src/main/java/com/ard/framework/web/service/SysLoginService.java
@@ -122,7 +122,7 @@
         if (!soloLogin)
         {
             // 如果用户不允许多终端同时登录,清除缓存信息
-            String userIdKey = Constants.LOGIN_USERID_KEY + loginUser.getUser().getUserId();
+            String userIdKey = CacheConstants.LOGIN_USERID_KEY + loginUser.getUser().getUserId();
             String userKey = redisCache.getCacheObject(userIdKey);
             if (StringUtils.isNotEmpty(userKey))
             {
diff --git a/ard-framework/src/main/java/com/ard/framework/web/service/TokenService.java b/ard-framework/src/main/java/com/ard/framework/web/service/TokenService.java
index a49ca2d..682a1ab 100644
--- a/ard-framework/src/main/java/com/ard/framework/web/service/TokenService.java
+++ b/ard-framework/src/main/java/com/ard/framework/web/service/TokenService.java
@@ -246,7 +246,7 @@
 
     private String getUserIdKey(Long userId)
     {
-        return Constants.LOGIN_USERID_KEY + userId;
+        return CacheConstants.LOGIN_USERID_KEY + userId;
     }
 
     public LoginUser getLoginUserByUserId(Long userId) {

--
Gitblit v1.9.3