diff --git a/src/main/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplate.java b/src/main/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplate.java index b1a8688..213c4b3 100644 --- a/src/main/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplate.java +++ b/src/main/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplate.java @@ -51,6 +51,7 @@ public class RedisCacheEnhanceTemplate { } public void set(String key, Object value, long ttlSeconds) { + requirePositiveTtl(ttlSeconds); redis.opsForValue().set(keyBuilder.build(key), codec.encode(value), applyJitter(ttlSeconds), TimeUnit.SECONDS); } @@ -94,6 +95,7 @@ public class RedisCacheEnhanceTemplate { // ---- 续期 ---- public boolean expire(String key, long ttlSeconds) { + requirePositiveTtl(ttlSeconds); return Boolean.TRUE.equals( redis.expire(keyBuilder.build(key), applyJitter(ttlSeconds), TimeUnit.SECONDS)); } @@ -125,6 +127,13 @@ public class RedisCacheEnhanceTemplate { return ttlSeconds + ThreadLocalRandom.current().nextLong(bound + 1); } + /** TTL 必须为正:0/负会被 Redis 当作"立即删除"(EXPIRE)或非法过期时间(SET),显式 fail-fast。 */ + private static void requirePositiveTtl(long ttlSeconds) { + if (ttlSeconds <= 0) { + throw new IllegalArgumentException("ttlSeconds must be positive, but was " + ttlSeconds); + } + } + // ---- 读取或回源(三防核心)---- public T getOrLoad(String key, Class type, Supplier loader) { @@ -140,6 +149,7 @@ public class RedisCacheEnhanceTemplate { } private T doGetOrLoad(String key, Function decoder, long ttlSeconds, Supplier loader) { + requirePositiveTtl(ttlSeconds); String dataKey = keyBuilder.build(key); Lookup r = tryGet(dataKey, decoder); diff --git a/src/test/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplateTest.java b/src/test/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplateTest.java index c1b0bba..e5c7296 100644 --- a/src/test/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplateTest.java +++ b/src/test/java/com/njcn/middle/cache/template/RedisCacheEnhanceTemplateTest.java @@ -240,4 +240,28 @@ class RedisCacheEnhanceTemplateTest { User got = template.getOrLoad("user:1", User.class, 60, () -> new User(1L, "fresh")); assertEquals("fresh", got.getName()); } + + // ---- #1+#2: ttl<=0 入口校验(fail-fast,不下发非法过期) ---- + + @Test + void setRejectsNonPositiveTtl() { + User u = new User(1L, "a"); + assertThrows(IllegalArgumentException.class, () -> template.set("user:1", u, 0)); + assertThrows(IllegalArgumentException.class, () -> template.set("user:1", u, -1)); + verify(valueOps, never()).set(anyString(), anyString(), anyLong(), any()); + } + + @Test + void expireRejectsNonPositiveTtl() { + assertThrows(IllegalArgumentException.class, () -> template.expire("user:1", 0)); + assertThrows(IllegalArgumentException.class, () -> template.expire("user:1", -5)); + verify(redis, never()).expire(anyString(), anyLong(), any()); + } + + @Test + void getOrLoadRejectsNonPositiveTtl() { + props.getBreakdown().setEnabled(false); // 避免跑红时走入等待自旋 + assertThrows(IllegalArgumentException.class, + () -> template.getOrLoad("user:1", User.class, 0, () -> new User(1L, "a"))); + } }