shishengliang/lingniu-vehicle-ingest
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: remove legacy session store enum

Browse Source
This commit is contained in:
lingniu
2026-07-01 10:35:12 +08:00
parent 87830fee09
commit e2b6ab63d6
2 changed files with 14 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
modules/core/session-core/src/main/java/com/lingniu/ingest/session/SessionProperties.java
View File

@@ -3,23 +3,25 @@ package com.lingniu.ingest.session;
import org.springframework.boot.context.properties.ConfigurationProperties; import org.springframework.boot.context.properties.ConfigurationProperties;
import java.time.Duration; import java.time.Duration;
import java.util.Locale;
@ConfigurationProperties(prefix = "lingniu.ingest.session") @ConfigurationProperties(prefix = "lingniu.ingest.session")
public class SessionProperties { public class SessionProperties {
private Store store = Store.REDIS; private String store = "redis";
private Duration ttl = Duration.ofMinutes(30); private Duration ttl = Duration.ofMinutes(30);
public Store getStore() { public String getStore() {
return store; return store;
} }
public void setStore(Store store) { public void setStore(String store) {
if (store == Store.MEMORY) { String value = store == null || store.isBlank() ? "redis" : store.trim().toLowerCase(Locale.ROOT);
throw new IllegalStateException("session.store supports only redis; configured value: memory"); if (!"redis".equals(value)) {
throw new IllegalStateException("session.store supports only redis; configured value: " + store);
} }
this.store = store; this.store = value;
} }
public Duration getTtl() { public Duration getTtl() {
@@ -30,8 +32,4 @@ public class SessionProperties {
this.ttl = ttl; this.ttl = ttl;
} }
public enum Store {
MEMORY,
REDIS
}
} }

6
modules/core/session-core/src/test/java/com/lingniu/ingest/session/config/SessionCoreAutoConfigurationTest.java
View File

@@ -61,4 +61,10 @@ class SessionCoreAutoConfigurationTest {
.hasStackTraceContaining("session.store supports only redis") .hasStackTraceContaining("session.store supports only redis")
.hasStackTraceContaining("memory")); .hasStackTraceContaining("memory"));
} }
@Test
void sessionPropertiesDoNotExposeLegacyStoreEnum() {
assertThatThrownBy(() -> Class.forName("com.lingniu.ingest.session.SessionProperties$Store"))
.isInstanceOf(ClassNotFoundException.class);
}
} }