修复JschSessionPool.remove逻辑错误问题。

CHANGELOG.md

@@ -11,6 +11,7 @@
 
 ### 🐞Bug修复
 * 【jwt    】      修复verify方法在定义alg为`none`时验证失效问题（issue#4105@Github）
+* 【extra  】      修复`JschSessionPool.remove`逻辑错误问题。
 
 -------------------------------------------------------------------------------------------------------------
 # 5.8.41(2025-10-12)

hutool-core/src/test/java/cn/hutool/core/lang/SimpleCacheTest.java

@@ -6,8 +6,9 @@ import cn.hutool.core.thread.ThreadUtil;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
+import java.util.Map;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import static org.junit.jupiter.api.Assertions.*;
 
 public class SimpleCacheTest {
 
@@ -61,4 +62,36 @@ public class SimpleCacheTest {
 		assertEquals("aaaValue", cache.get("aaa"));
 		IoUtil.close(tester);
 	}
+
+	@Test
+	void removeTest(){
+		final SimpleCache<String, String> cache = new SimpleCache<>();
+		cache.put("key1", "value1");
+		cache.get("key1");
+		cache.put("key2", "value2");
+		cache.get("key2");
+		cache.put("key3", "value3");
+		cache.get("key3");
+		cache.put("key4", "value4");
+		cache.get("key4");
+		cache.get("key5", ()->"value5");
+
+		String key = null;
+		for (Map.Entry<String, String> entry : cache) {
+			if ("value3".equals(entry.getValue())) {
+				key = entry.getKey();
+				break;
+			}
+		}
+
+		if(null != key){
+			cache.remove(key);
+		}
+
+		assertEquals("value1", cache.get("key1"));
+		assertEquals("value2", cache.get("key2"));
+		assertEquals("value4", cache.get("key4"));
+		assertEquals("value5", cache.get("key5"));
+		assertNull(cache.get("key3"));
+	}
 }

hutool-extra/src/main/java/cn/hutool/extra/ssh/JschSessionPool.java

@@ -4,7 +4,7 @@ import cn.hutool.core.lang.SimpleCache;
 import cn.hutool.core.util.StrUtil;
 import com.jcraft.jsch.Session;
 
-import java.util.Iterator;
+import java.util.Map;
 import java.util.Map.Entry;
 
 /**
@@ -110,15 +110,17 @@ public enum JschSessionPool {
 	 */
 	public void remove(Session session) {
 		if (null != session) {
-			final Iterator<Entry<String, Session>> iterator = this.cache.iterator();
+			String key = null;
-			Entry<String, Session> entry;
+			for (Map.Entry<String, Session> entry : cache) {
-			while (iterator.hasNext()) {
-				entry = iterator.next();
 				if (session.equals(entry.getValue())) {
-					iterator.remove();
+					key = entry.getKey();
 					break;
 				}
 			}
+
+			if(null != key){
+				cache.remove(key);
+			}
 		}
 	}