Add logging for authentication context and cleanup debug logs #deploy…

…-idporten-frontend Added logging for the authentication context in `JwtResolver` to enhance traceability. Removed redundant debug logs of exchange attributes in multiple classes for clarity and to reduce noise in the logs.
navikt · Dec 17, 2024 · 3ba51fc · 3ba51fc
1 parent a757313
commit 3ba51fc
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 9 deletions.
diff --git a/...nd/src/main/java/no/nav/dolly/web/consumers/Altinn3PersonOrganisasjonTilgangConsumer.java b/...nd/src/main/java/no/nav/dolly/web/consumers/Altinn3PersonOrganisasjonTilgangConsumer.java
@@ -40,9 +40,6 @@ public Altinn3PersonOrganisasjonTilgangConsumer(
 
     public Mono<Boolean> hasAccess(String organisasjonsnummer, ServerWebExchange exchange) {
 
-        exchange.getAttributes()
-                .forEach((key, value) -> log.info("Atributt {}: {}", key, value));
-
         return getAuthenticatedUserId
                 .call()
                 .flatMap(userId -> accessService.getAccessToken(serverProperties, exchange)
@@ -59,12 +56,10 @@ public Mono<Boolean> hasAccess(String organisasjonsnummer, ServerWebExchange exc
 
     public Flux<OrganisasjonDTO> getOrganisasjoner(ServerWebExchange exchange) {
 
-        exchange.getAttributes()
-                .forEach((key, value) -> log.info("Atributt {}: {}", key, value));
-
         return getAuthenticatedUserId
                 .call()
                 .flatMapMany(userId -> accessService.getAccessToken(serverProperties, exchange)
+                        .doOnNext(pid -> log.info("userid {}", pid))
                         .flatMapMany(accessToken -> new PostPersonOrganisasjonTilgangCommand(webClient, userId, accessToken).call()));
     }
 }

diff --git a/apps/dolly-frontend/src/main/java/no/nav/dolly/web/provider/web/BrukerTilgangController.java b/apps/dolly-frontend/src/main/java/no/nav/dolly/web/provider/web/BrukerTilgangController.java
@@ -23,9 +23,6 @@ public class BrukerTilgangController {
     @GetMapping("/organisasjoner")
     public Mono<List<OrganisasjonDTO>> getOrganisasjoner(ServerWebExchange exchange) {
 
-        exchange.getAttributes()
-                .forEach((key, value) -> log.info("Atributt {}: {}", key, value));
-
         return altinn3PersonOrganisasjonTilgangConsumer.getOrganisasjoner(exchange)
                 .collectList();
     }

diff --git a/...ctive-security/src/main/java/no/nav/testnav/libs/reactivesecurity/action/JwtResolver.java b/...ctive-security/src/main/java/no/nav/testnav/libs/reactivesecurity/action/JwtResolver.java
@@ -19,6 +19,7 @@ Mono<JwtAuthenticationToken> getJwtAuthenticationToken() {
         return ReactiveSecurityContextHolder
                 .getContext()
                 .switchIfEmpty(Mono.error(new JwtResolverException("ReactiveSecurityContext is empty")))
+                .doOnNext(context -> log.info("context.authentication {}", context.getAuthentication()))
                 .map(SecurityContext::getAuthentication)
                 .map(JwtAuthenticationToken.class::cast)
                 .doOnError(throwable -> log.warn("Klarte ikke hente Jwt Auth Token", throwable))