Skip to content

Commit c08f66a

Browse files
committed
Add warning for invalid HA configs
1 parent 59ac372 commit c08f66a

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

src/main/java/eu/openanalytics/containerproxy/ContainerProxyApplication.java

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,8 @@
7676
import java.util.Properties;
7777
import java.util.concurrent.Executor;
7878

79+
import static eu.openanalytics.containerproxy.service.ProxyService.PROPERTY_STOP_PROXIES_ON_SHUTDOWN;
80+
7981
@EnableScheduling
8082
@EnableAsync
8183
@SpringBootApplication(exclude = {UserDetailsServiceAutoConfiguration.class})
@@ -148,6 +150,19 @@ public void init() {
148150
if (sameSiteCookiePolicy.equalsIgnoreCase("none") && !secureCookiesEnabled) {
149151
log.warn("WARNING: Invalid configuration detected: same-site-cookie policy is set to None, but secure-cookies are not enabled. Secure cookies must be enabled when using None as same-site-cookie policy ");
150152
}
153+
154+
155+
if (environment.getProperty("proxy.store-mode", "").equalsIgnoreCase("Redis")) {
156+
if (!environment.getProperty("spring.session.store-type", "").equalsIgnoreCase("redis")) {
157+
// running in HA mode, but not using Redis sessions
158+
log.warn("WARNING: Invalid configuration detected: store-mode is set to Redis (i.e. High-Availability mode), but you are not using Redis for user sessions!");
159+
}
160+
if (environment.getProperty(PROPERTY_STOP_PROXIES_ON_SHUTDOWN, Boolean.class, true)) {
161+
// running in HA mode, but proxies are removed when shutting down
162+
log.warn("WARNING: Invalid configuration detected: store-mode is set to Redis (i.e. High-Availability mode), but proxies are stopped at shutdown of server!");
163+
}
164+
}
165+
151166
}
152167

153168
@Autowired(required = false)

0 commit comments

Comments
 (0)