Skip to content

Commit de36bbe

Browse files
committed
Additional logging.
1 parent 7d1512e commit de36bbe

File tree

4 files changed

+15
-20
lines changed

4 files changed

+15
-20
lines changed

libs/reactive-security/src/main/java/no/nav/testnav/libs/reactivesecurity/action/GetAuthenticatedResourceServerType.java

+11-15
Original file line numberDiff line numberDiff line change
@@ -22,21 +22,17 @@ public class GetAuthenticatedResourceServerType extends JwtResolver implements C
2222
private Optional<ResourceServerType> getResourceTypeFrom(JwtAuthenticationToken token) {
2323
return resourceServerProperties
2424
.stream()
25-
.filter(properties -> {
26-
if (token == null) {
27-
log.warn("Token is null");
28-
} else if (token.getToken() == null) {
29-
log.warn("Token.getToken() is null");
30-
} else if (token.getToken().getIssuer() == null) {
31-
log.warn("Token.getToken().getIssuer() is null");
32-
}
33-
return Optional
34-
.ofNullable(token)
35-
.map(JwtAuthenticationToken::getToken)
36-
.map(JwtClaimAccessor::getIssuer)
37-
.map(issuerFromToken -> issuerFromToken.toString().equalsIgnoreCase(properties.getIssuerUri()))
38-
.orElse(false);
39-
})
25+
.filter(properties ->
26+
Optional
27+
.ofNullable(token)
28+
.map(JwtAuthenticationToken::getToken)
29+
.map(JwtClaimAccessor::getIssuer)
30+
.map(issuerFromToken -> {
31+
var issuer = issuerFromToken.toString().equalsIgnoreCase(properties.getIssuerUri());
32+
log.info("issuerFromToken: {}, properties.getIssuerUri(): {}, issuer: {}", issuerFromToken, properties.getIssuerUri(), issuer);
33+
return issuer;
34+
})
35+
.orElse(false))
4036
.findFirst()
4137
.map(ResourceServerProperties::getType);
4238
}

libs/reactive-security/src/main/java/no/nav/testnav/libs/reactivesecurity/action/JwtResolver.java

+2-3
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,14 @@ Mono<Authentication> getJwtAuthenticationToken() {
1414
.getContext()
1515
.switchIfEmpty(Mono.error(new EmptyReactiveSecurityContextException()))
1616
.doOnNext(context -> log.info("JwtResolver context.authentication {} {}", context.getAuthentication().getClass().getCanonicalName(), context.getAuthentication()))
17-
.map(SecurityContext::getAuthentication);
17+
.map(SecurityContext::getAuthentication)
18+
.doOnNext(authentication -> log.info("JwtResolver authentication {} {}", authentication.getClass().getCanonicalName(), authentication));
1819
}
1920

2021
static class EmptyReactiveSecurityContextException extends IllegalStateException {
21-
2222
EmptyReactiveSecurityContextException() {
2323
super();
2424
}
25-
2625
}
2726

2827
}

libs/servlet-security/src/main/java/no/nav/testnav/libs/servletsecurity/jwt/MultipleIssuersJwtDecoder.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ private static OAuth2TokenValidator<Jwt> customAudienceValidator(ResourceServerP
6565
.stream()
6666
.anyMatch(audience -> properties.getAcceptedAudience().contains(audience));
6767
if (!valid) {
68-
log.error("Fant ikke påkrevd audience {} i tokenet, bare {}", properties.getAcceptedAudience(), token.getAudience());
68+
log.warn("Fant ikke påkrevd audience {} i tokenet, bare {}", properties.getAcceptedAudience(), token.getAudience());
6969
}
7070
return OAuth2TokenValidatorResult.success();
7171
};

libs/servlet-security/src/main/java/no/nav/testnav/libs/servletsecurity/jwt/SecureOAuth2ServerToServerAutoConfiguration.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public class SecureOAuth2ServerToServerAutoConfiguration {
3636
@Bean
3737
@Profile("!test")
3838
@ConditionalOnMissingBean
39-
JwtDecoder jwtDecoder(List<ResourceServerProperties> properties) {
39+
MultipleIssuersJwtDecoder jwtDecoder(List<ResourceServerProperties> properties) {
4040
return new MultipleIssuersJwtDecoder(properties);
4141
}
4242

0 commit comments

Comments
 (0)