Skip to content

Commit

Permalink
Merge pull request #5 from OSGP/feature/FDP-94-actuator
Browse files Browse the repository at this point in the history
FDP-94: Actuator fix
  • Loading branch information
sanderv authored Nov 22, 2023
2 parents a645dd5 + cd36f45 commit 29bd978
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 2 deletions.
3 changes: 3 additions & 0 deletions application/build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,9 @@ dependencies {
testImplementation("org.junit.jupiter:junit-jupiter-engine")
testImplementation("org.junit.jupiter:junit-jupiter-params")
testImplementation("org.mockito:mockito-junit-jupiter")

// Generate test and integration test reports
jacocoAggregation(project(":application"))
}

tasks.withType<org.springframework.boot.gradle.tasks.bundling.BootBuildImage> {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,11 @@ public SoapEndpointMapping(final SoapEndpoint soapEndpoint) {

@Override
protected Object getHandlerInternal(@NotNull final HttpServletRequest request) {
return soapEndpoint;
if (request.getServletPath().startsWith("/actuator")) {
// Let Spring handle this routing
return null;
} else {
return soapEndpoint;
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ class SecurityConfiguration {
fun filterChain(http: HttpSecurity): SecurityFilterChain =
http.authorizeHttpRequests {
it
.requestMatchers("/actuator/**").permitAll()
.anyRequest().authenticated()
}.x509 {
it
Expand Down
12 changes: 11 additions & 1 deletion application/src/main/resources/application-dev.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ logging:
org:
gxf:
soapbridge: DEBUG

spring:
kafka:
bootstrap-servers: localhost:9092
Expand Down Expand Up @@ -46,3 +45,14 @@ soap:
port: 443
protocol: https
time-out: 45

management:
server:
port: 8888
# ssl:
# enabled: false
# client-auth: none
endpoints:
web:
exposure:
include: health

0 comments on commit 29bd978

Please sign in to comment.