Skip to content

Commit 7093602

Browse files
committed
Simplify UserDetailsService creation in samples
Closes gh-10385
1 parent 61bbaf1 commit 7093602

File tree

9 files changed

+17
-49
lines changed

9 files changed

+17
-49
lines changed

spring-boot-dependencies/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@
176176
<spring-plugin.version>1.2.0.RELEASE</spring-plugin.version>
177177
<spring-restdocs.version>1.2.1.RELEASE</spring-restdocs.version>
178178
<spring-retry.version>1.2.1.RELEASE</spring-retry.version>
179-
<spring-security.version>5.0.0.M4</spring-security.version>
179+
<spring-security.version>5.0.0.BUILD-SNAPSHOT</spring-security.version>
180180
<spring-session.version>2.0.0.M4</spring-session.version>
181181
<spring-session-data-mongodb.version>2.0.0.M3</spring-session-data-mongodb.version>
182182
<spring-social.version>2.0.0.M4</spring-social.version>

spring-boot-integration-tests/spring-boot-security-tests/spring-boot-security-test-web-helloworld/src/main/java/sample/HelloWebSecurityApplication.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,18 +20,14 @@
2020
import org.springframework.boot.autoconfigure.SpringBootApplication;
2121
import org.springframework.context.annotation.Bean;
2222
import org.springframework.security.core.userdetails.User;
23-
import org.springframework.security.core.userdetails.UserDetailsService;
2423
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
2524

2625
@SpringBootApplication
2726
public class HelloWebSecurityApplication {
2827

2928
@Bean
30-
public UserDetailsService userDetailsService() throws Exception {
31-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
32-
manager.createUser(
33-
User.withUsername("user").password("password").roles("USER").build());
34-
return manager;
29+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
30+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
3531
}
3632

3733
public static void main(String[] args) {

spring-boot-samples/spring-boot-sample-actuator-log4j2/src/main/java/sample/actuator/log4j2/SampleActuatorLog4J2Application.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,18 +20,14 @@
2020
import org.springframework.boot.autoconfigure.SpringBootApplication;
2121
import org.springframework.context.annotation.Bean;
2222
import org.springframework.security.core.userdetails.User;
23-
import org.springframework.security.core.userdetails.UserDetailsService;
2423
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
2524

2625
@SpringBootApplication
2726
public class SampleActuatorLog4J2Application {
2827

2928
@Bean
30-
public UserDetailsService userDetailsService() throws Exception {
31-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
32-
manager.createUser(
33-
User.withUsername("user").password("password").roles("USER").build());
34-
return manager;
29+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
30+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
3531
}
3632

3733
public static void main(String[] args) throws Exception {

spring-boot-samples/spring-boot-sample-actuator-ui/src/main/java/sample/actuator/ui/SampleActuatorUiApplication.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
import org.springframework.boot.autoconfigure.SpringBootApplication;
2424
import org.springframework.context.annotation.Bean;
2525
import org.springframework.security.core.userdetails.User;
26-
import org.springframework.security.core.userdetails.UserDetailsService;
2726
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
2827
import org.springframework.stereotype.Controller;
2928
import org.springframework.web.bind.annotation.GetMapping;
@@ -34,11 +33,8 @@
3433
public class SampleActuatorUiApplication {
3534

3635
@Bean
37-
public UserDetailsService userDetailsService() throws Exception {
38-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
39-
manager.createUser(
40-
User.withUsername("user").password("password").roles("USER").build());
41-
return manager;
36+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
37+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
4238
}
4339

4440
@GetMapping("/")

spring-boot-samples/spring-boot-sample-actuator/src/main/java/sample/actuator/SampleActuatorApplication.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,6 @@
2323
import org.springframework.boot.context.properties.EnableConfigurationProperties;
2424
import org.springframework.context.annotation.Bean;
2525
import org.springframework.security.core.userdetails.User;
26-
import org.springframework.security.core.userdetails.UserDetailsService;
2726
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
2827

2928
@SpringBootApplication
@@ -35,11 +34,8 @@ public static void main(String[] args) throws Exception {
3534
}
3635

3736
@Bean
38-
public UserDetailsService userDetailsService() throws Exception {
39-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
40-
manager.createUser(
41-
User.withUsername("user").password("password").roles("USER").build());
42-
return manager;
37+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
38+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
4339
}
4440

4541
@Bean

spring-boot-samples/spring-boot-sample-secure/src/main/java/sample/secure/SampleSecureApplication.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@
2727
import org.springframework.security.core.authority.AuthorityUtils;
2828
import org.springframework.security.core.context.SecurityContextHolder;
2929
import org.springframework.security.core.userdetails.User;
30-
import org.springframework.security.core.userdetails.UserDetailsService;
3130
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
3231

3332
@EnableAutoConfiguration
@@ -39,11 +38,8 @@ public class SampleSecureApplication implements CommandLineRunner {
3938
private SampleService service;
4039

4140
@Bean
42-
public UserDetailsService userDetailsService() throws Exception {
43-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
44-
manager.createUser(
45-
User.withUsername("user").password("password").roles("USER").build());
46-
return manager;
41+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
42+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
4743
}
4844

4945
@Override

spring-boot-samples/spring-boot-sample-servlet/src/main/java/sample/servlet/SampleServletApplication.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -31,19 +31,15 @@
3131
import org.springframework.boot.web.servlet.support.SpringBootServletInitializer;
3232
import org.springframework.context.annotation.Bean;
3333
import org.springframework.security.core.userdetails.User;
34-
import org.springframework.security.core.userdetails.UserDetailsService;
3534
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
3635

3736
@SpringBootConfiguration
3837
@EnableAutoConfiguration
3938
public class SampleServletApplication extends SpringBootServletInitializer {
4039

4140
@Bean
42-
public UserDetailsService userDetailsService() throws Exception {
43-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
44-
manager.createUser(
45-
User.withUsername("user").password("password").roles("USER").build());
46-
return manager;
41+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
42+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
4743
}
4844

4945
@SuppressWarnings("serial")

spring-boot-samples/spring-boot-sample-session/src/main/java/sample/session/SampleSessionApplication.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
import org.springframework.boot.autoconfigure.SpringBootApplication;
2121
import org.springframework.context.annotation.Bean;
2222
import org.springframework.security.core.userdetails.User;
23-
import org.springframework.security.core.userdetails.UserDetailsService;
2423
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
2524

2625
@SpringBootApplication
@@ -31,11 +30,8 @@ public static void main(String[] args) throws Exception {
3130
}
3231

3332
@Bean
34-
public UserDetailsService userDetailsService() {
35-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
36-
manager.createUser(
37-
User.withUsername("user").password("password").roles("USER").build());
38-
return manager;
33+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
34+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
3935
}
4036

4137
}

spring-boot-test-autoconfigure/src/test/java/org/springframework/boot/test/autoconfigure/security/SecurityTestApplication.java

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
import org.springframework.context.annotation.Bean;
2222
import org.springframework.security.access.annotation.Secured;
2323
import org.springframework.security.core.userdetails.User;
24-
import org.springframework.security.core.userdetails.UserDetailsService;
2524
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
2625
import org.springframework.web.bind.annotation.RequestMapping;
2726
import org.springframework.web.bind.annotation.RestController;
@@ -35,11 +34,8 @@
3534
public class SecurityTestApplication {
3635

3736
@Bean
38-
public UserDetailsService userDetailsService() throws Exception {
39-
InMemoryUserDetailsManager manager = new InMemoryUserDetailsManager();
40-
manager.createUser(
41-
User.withUsername("user").password("secret").roles("USER").build());
42-
return manager;
37+
public InMemoryUserDetailsManager inMemoryUserDetailsManager() throws Exception {
38+
return new InMemoryUserDetailsManager(User.withUsername("user").password("password").roles("USER").build());
4339
}
4440

4541
@RestController

0 commit comments

Comments
 (0)