Skip to content

Commit 771c306

Browse files
author
Ivan Franchin
committed
Project update
- Rename WebSecurityConfig to SecurityConfig; - Remove EnableWebSecurity annotation as it is not needed; - To create the PasswordEncoder Bean, using PasswordEncoderFactories.createDelegatingPasswordEncoder(); instead of new BCryptPasswordEncoder(); - Remove Actuator dependency; - Remove public keyword from Beans.
1 parent e97a053 commit 771c306

File tree

7 files changed

+9
-42
lines changed

7 files changed

+9
-42
lines changed

bitcoin-api/pom.xml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,6 @@
2929
<spring-cloud.version>2024.0.0</spring-cloud.version>
3030
</properties>
3131
<dependencies>
32-
<dependency>
33-
<groupId>org.springframework.boot</groupId>
34-
<artifactId>spring-boot-starter-actuator</artifactId>
35-
</dependency>
3632
<dependency>
3733
<groupId>org.springframework.boot</groupId>
3834
<artifactId>spring-boot-starter-data-jpa</artifactId>

bitcoin-api/src/main/java/com/ivanfranchin/bitcoinapi/config/SwaggerConfig.java

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,12 @@ public class SwaggerConfig {
1515
private String applicationName;
1616

1717
@Bean
18-
public OpenAPI customOpenAPI() {
18+
OpenAPI customOpenAPI() {
1919
return new OpenAPI().components(new Components()).info(new Info().title(applicationName));
2020
}
2121

2222
@Bean
23-
public GroupedOpenApi customApi() {
23+
GroupedOpenApi customApi() {
2424
return GroupedOpenApi.builder().group("api").pathsToMatch("/api/**").build();
2525
}
26-
27-
@Bean
28-
public GroupedOpenApi actuatorApi() {
29-
return GroupedOpenApi.builder().group("actuator").pathsToMatch("/actuator/**").build();
30-
}
3126
}

bitcoin-api/src/main/resources/application.yml

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -17,16 +17,6 @@ spring:
1717
prices-out-0:
1818
destination: com.ivanfranchin.bitcoin.api.price
1919

20-
management:
21-
endpoints:
22-
web:
23-
exposure.include: beans, env, health, info, metrics, mappings
24-
endpoint:
25-
health:
26-
show-details: always
27-
2820
springdoc:
29-
show-actuator: true
3021
swagger-ui:
31-
groups-order: DESC
3222
disable-swagger-default-url: true

bitcoin-client/pom.xml

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,6 @@
2828
<spring-cloud.version>2024.0.0</spring-cloud.version>
2929
</properties>
3030
<dependencies>
31-
<dependency>
32-
<groupId>org.springframework.boot</groupId>
33-
<artifactId>spring-boot-starter-actuator</artifactId>
34-
</dependency>
3531
<dependency>
3632
<groupId>org.springframework.boot</groupId>
3733
<artifactId>spring-boot-starter-security</artifactId>

bitcoin-client/src/main/java/com/ivanfranchin/bitcoinclient/kafka/PriceStream.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ public class PriceStream {
2121
private final SimpMessagingTemplate simpMessagingTemplate;
2222

2323
@Bean
24-
public Consumer<Message<PriceMessage>> prices() {
24+
Consumer<Message<PriceMessage>> prices() {
2525
return message -> {
2626
PriceMessage priceMessage = message.getPayload();
2727
MessageHeaders messageHeaders = message.getHeaders();

bitcoin-client/src/main/java/com/ivanfranchin/bitcoinclient/security/WebSecurityConfig.java renamed to bitcoin-client/src/main/java/com/ivanfranchin/bitcoinclient/security/SecurityConfig.java

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,21 @@
44
import org.springframework.context.annotation.Configuration;
55
import org.springframework.security.config.Customizer;
66
import org.springframework.security.config.annotation.web.builders.HttpSecurity;
7-
import org.springframework.security.config.annotation.web.configuration.EnableWebSecurity;
87
import org.springframework.security.core.userdetails.User;
98
import org.springframework.security.core.userdetails.UserDetails;
109
import org.springframework.security.core.userdetails.UserDetailsService;
11-
import org.springframework.security.crypto.bcrypt.BCryptPasswordEncoder;
10+
import org.springframework.security.crypto.factory.PasswordEncoderFactories;
1211
import org.springframework.security.crypto.password.PasswordEncoder;
1312
import org.springframework.security.provisioning.InMemoryUserDetailsManager;
1413
import org.springframework.security.web.SecurityFilterChain;
1514

1615
@Configuration
17-
@EnableWebSecurity
18-
public class WebSecurityConfig {
16+
public class SecurityConfig {
1917

2018
private static final String USER = "USER";
2119

2220
@Bean
23-
public UserDetailsService userDetailsService() {
21+
UserDetailsService userDetailsService() {
2422
UserDetails user1 = User.withUsername("user1")
2523
.password(passwordEncoder().encode("123"))
2624
.roles(USER).build();
@@ -33,15 +31,15 @@ public UserDetailsService userDetailsService() {
3331
}
3432

3533
@Bean
36-
public SecurityFilterChain securityFilterChain(HttpSecurity http) throws Exception {
34+
SecurityFilterChain securityFilterChain(HttpSecurity http) throws Exception {
3735
return http.authorizeHttpRequests(authorizeHttpRequests -> authorizeHttpRequests
3836
.anyRequest().authenticated())
3937
.formLogin(Customizer.withDefaults())
4038
.build();
4139
}
4240

4341
@Bean
44-
public PasswordEncoder passwordEncoder() {
45-
return new BCryptPasswordEncoder();
42+
PasswordEncoder passwordEncoder() {
43+
return PasswordEncoderFactories.createDelegatingPasswordEncoder();
4644
}
4745
}

bitcoin-client/src/main/resources/application.yml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -10,11 +10,3 @@ spring:
1010
prices-in-0:
1111
destination: com.ivanfranchin.bitcoin.api.price
1212
group: bitcoinClientGroup
13-
14-
management:
15-
endpoints:
16-
web:
17-
exposure.include: beans, env, health, info, metrics, mappings
18-
endpoint:
19-
health:
20-
show-details: always

0 commit comments

Comments
 (0)