Skip to content

Commit cd42a94

Browse files
committed
Make cleanup cron configurable
1 parent 6214dfc commit cd42a94

File tree

3 files changed

+2
-18
lines changed

3 files changed

+2
-18
lines changed

orcid-web/src/main/java/org/orcid/frontend/spring/session/redis/OrcidEnableRedisHttpSession.java

-2
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,5 @@
3232

3333
FlushMode flushMode() default FlushMode.ON_SAVE;
3434

35-
String cleanupCron() default "0 * * * * *";
36-
3735
SaveMode saveMode() default SaveMode.ON_SET_ATTRIBUTE;
3836
}

orcid-web/src/main/java/org/orcid/frontend/spring/session/redis/OrcidRedisHttpSessionConfiguration.java

+2-13
Original file line numberDiff line numberDiff line change
@@ -64,8 +64,6 @@
6464
)
6565
@Profile("!unitTests")
6666
public class OrcidRedisHttpSessionConfiguration extends SpringHttpSessionConfiguration implements BeanClassLoaderAware, EmbeddedValueResolverAware, ImportAware {
67-
68-
static final String DEFAULT_CLEANUP_CRON = "0 * * * * *";
6967
private Integer maxInactiveIntervalInSeconds = 1800;
7068
private String redisNamespace = "spring:session";
7169
private FlushMode flushMode;
@@ -82,10 +80,10 @@ public class OrcidRedisHttpSessionConfiguration extends SpringHttpSessionConfigu
8280
private ClassLoader classLoader;
8381
private StringValueResolver embeddedValueResolver;
8482

85-
public OrcidRedisHttpSessionConfiguration() {
83+
public OrcidRedisHttpSessionConfiguration(@Value("${org.orcid.core.utils.cache.session.redis.cleanup.cron:5 */5 * * * *}") String cleanupCron) {
8684
this.flushMode = FlushMode.ON_SAVE;
8785
this.saveMode = SaveMode.ON_SET_ATTRIBUTE;
88-
this.cleanupCron = "0 * * * * *";
86+
this.cleanupCron = cleanupCron;
8987
this.configureRedisAction = new ConfigureNotifyKeyspaceEventsAction();
9088
}
9189

@@ -163,10 +161,6 @@ public void setSaveMode(SaveMode saveMode) {
163161
this.saveMode = saveMode;
164162
}
165163

166-
public void setCleanupCron(String cleanupCron) {
167-
this.cleanupCron = cleanupCron;
168-
}
169-
170164
@Autowired(
171165
required = false
172166
)
@@ -252,11 +246,6 @@ public void setImportMetadata(AnnotationMetadata importMetadata) {
252246

253247
this.flushMode = flushMode;
254248
this.saveMode = (SaveMode)attributes.getEnum("saveMode");
255-
String cleanupCron = attributes.getString("cleanupCron");
256-
if (StringUtils.hasText(cleanupCron)) {
257-
this.cleanupCron = cleanupCron;
258-
}
259-
260249
}
261250

262251
private RedisTemplate<Object, Object> createRedisTemplate() {

orcid-web/src/main/java/org/orcid/frontend/web/util/SwitchUserGrantedAuthorityDeserializer.java

-3
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,6 @@ public SwitchUserGrantedAuthority deserialize(JsonParser p, DeserializationConte
2222
String role = authorityNode.asText();
2323
UsernamePasswordAuthenticationToken authentication = mapper.convertValue(sourceNode, UsernamePasswordAuthenticationToken.class);
2424

25-
System.out.println(sourceNode.asText());
26-
System.out.println(authorityNode.asText());
27-
2825
return new SwitchUserGrantedAuthority(role, authentication);
2926
}
3027
}

0 commit comments

Comments
 (0)