Skip to content

Commit d44f8b1

Browse files
committed
Circular reference when injecting SessionRegistry #1455
1 parent f24f06e commit d44f8b1

1 file changed

Lines changed: 4 additions & 10 deletions

File tree

jmix-security/security/src/main/java/io/jmix/security/SecurityConfiguration.java

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -64,12 +64,6 @@ public class SecurityConfiguration {
6464
@Autowired
6565
private RememberMeProperties rememberMeProperties;
6666

67-
@Autowired
68-
private PersistentTokenRepository rememberMeTokenRepository;
69-
70-
@Autowired
71-
private SessionRegistry sessionRegistry;
72-
7367
@Autowired
7468
private UserRepository userRepository;
7569

@@ -89,7 +83,7 @@ protected PersistentTokenRepository inMemoryRememberMeRepository() {
8983
}
9084

9185
@Bean("sec_rememberMeServices")
92-
public RememberMeServices rememberMeServices() {
86+
public RememberMeServices rememberMeServices(PersistentTokenRepository rememberMeTokenRepository) {
9387
JmixRememberMeServices rememberMeServices =
9488
new JmixRememberMeServices(rememberMeProperties.getKey(), userRepository, rememberMeTokenRepository);
9589
rememberMeServices.setTokenValiditySeconds(rememberMeProperties.getTokenValiditySeconds());
@@ -99,11 +93,11 @@ public RememberMeServices rememberMeServices() {
9993

10094
@Primary
10195
@Bean
102-
public SessionAuthenticationStrategy sessionControlAuthenticationStrategy() {
103-
return new CompositeSessionAuthenticationStrategy(strategies());
96+
public SessionAuthenticationStrategy sessionControlAuthenticationStrategy(SessionRegistry sessionRegistry) {
97+
return new CompositeSessionAuthenticationStrategy(strategies(sessionRegistry));
10498
}
10599

106-
protected List<SessionAuthenticationStrategy> strategies() {
100+
protected List<SessionAuthenticationStrategy> strategies(SessionRegistry sessionRegistry) {
107101
RegisterSessionAuthenticationStrategy registerSessionAuthenticationStrategy
108102
= new RegisterSessionAuthenticationStrategy(sessionRegistry);
109103
ConcurrentSessionControlAuthenticationStrategy concurrentSessionControlStrategy

0 commit comments

Comments
 (0)