Skip to content

Commit 7ad2b98

Browse files
authored
Revert "Add more logging in prod (#144)" (#145)
This reverts commit 607c15b.
1 parent 2128aa1 commit 7ad2b98

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

.nais/prod/nais.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -215,7 +215,6 @@ data:
215215
216216
logger:
217217
levels:
218-
root: INFO
219218
io.micronaut.security: INFO
220219
no.ssb.dlp.pseudo.service: INFO
221220
io.micronaut.security.token.jwt.validator: DEBUG

src/main/java/no/ssb/dlp/pseudo/service/security/CustomRolesFinder.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public List<String> resolveRoles(Map<String, Object> attributes) {
4444
email = Optional.ofNullable(Objects.toString(attributes.get(tokenConfiguration.getNameKey()), null));
4545
}
4646

47-
log.info("User {} has a trusted issuer? {}", email, trustedIssuer);
47+
log.debug("User {} has a trusted issuer? {}", email, trustedIssuer);
4848

4949
// We check for trustedIssuer when in environments where all authenticated requests are accepted
5050
// This is due to Google tokens being valid for authorization purposes,
@@ -67,16 +67,14 @@ public List<String> resolveRoles(Map<String, Object> attributes) {
6767
}
6868
if (rolesConfig.getUsersGroup().isPresent()) {
6969
final List<Membership> userMembers = cloudIdentityService.listMembers(rolesConfig.getUsersGroup().get());
70-
List<String> userEmails = cloudIdentityService.listMembers(rolesConfig.getUsersGroup().get()).stream().map(v -> v.preferredMemberKey().id()).toList();
71-
log.info("User group {} has members {}", rolesConfig.getUsersGroup().get(), userEmails);
7270
if (email.map(user_email -> userMembers.stream().anyMatch(value -> value.preferredMemberKey().id().equals(user_email))).orElse(false)) {
7371
roles.add(PseudoServiceRole.USER);
7472
}
7573
}
7674
if (roles.isEmpty()) {
7775
log.info("Could not resolve any roles for user {}", email);
7876
}
79-
log.info("Resolved roles {} for user {}", roles, email);
77+
log.debug("Resolved roles {} for user {}", roles, email);
8078
return roles;
8179
}
8280

0 commit comments

Comments
 (0)