diff --git a/src/main/java/ch/wisv/connect/authentication/CHAuthenticationProvider.java b/src/main/java/ch/wisv/connect/authentication/CHAuthenticationProvider.java index 57d8302..08bf559 100644 --- a/src/main/java/ch/wisv/connect/authentication/CHAuthenticationProvider.java +++ b/src/main/java/ch/wisv/connect/authentication/CHAuthenticationProvider.java @@ -63,8 +63,6 @@ public Authentication authenticate(Authentication authentication) throws Authent if (samlCredential.getRemoteEntityID().startsWith(SAML_GOOGLE_ENTITY_ID)) { log.info("Authenticated via Google SAML: email={}", samlCredential.getNameID().getValue()); - log.info("Remote entity ID: {}", samlCredential.getRemoteEntityID()); - log.info("SAML attributes: {}", attributes.stream().map(Attribute::getName).collect(Collectors.joining(", "))); CHUserDetails userDetails = userDetailService.loadUserByGoogleCredential(samlCredential); return CHAuthenticationToken.createAuthenticationToken(authentication, userDetails); } @@ -76,6 +74,8 @@ public Authentication authenticate(Authentication authentication) throws Authent String.format("%s=\"%s\"", n, samlCredential.getAttributeAsString(n))) .collect(Collectors.joining(" ")); log.info("Authenticated via SAML: netid={} {}", netid, attributesString); + } else { + log.info("Authenticated via SAML: netid={}", netid); } CHUserDetails userDetails; diff --git a/src/main/java/ch/wisv/connect/authentication/ExceptionMappingAuthenticationFailureHandler.java b/src/main/java/ch/wisv/connect/authentication/ExceptionMappingAuthenticationFailureHandler.java index d453b82..bff0dbe 100644 --- a/src/main/java/ch/wisv/connect/authentication/ExceptionMappingAuthenticationFailureHandler.java +++ b/src/main/java/ch/wisv/connect/authentication/ExceptionMappingAuthenticationFailureHandler.java @@ -39,9 +39,6 @@ public void onAuthenticationFailure(HttpServletRequest request, HttpServletRespo AuthenticationException exception) throws IOException, ServletException { String url = failureUrlMap.get(exception.getClass().getName()); - logger.info("Exception: " + exception.getClass().getName()); - logger.info("Message: " + exception.getMessage()); - if (url != null) { if (isUseForward()) { logger.debug("Authentication failure, forwarding to " + url); diff --git a/src/main/java/ch/wisv/connect/services/CHUserDetailsService.java b/src/main/java/ch/wisv/connect/services/CHUserDetailsService.java index a4cbd6e..4e318cc 100644 --- a/src/main/java/ch/wisv/connect/services/CHUserDetailsService.java +++ b/src/main/java/ch/wisv/connect/services/CHUserDetailsService.java @@ -214,9 +214,9 @@ private CHUserDetails createUserDetails(Person person, } String googleUsername = person.getGoogleUsername(); - System.out.println("googleUsername: " + googleUsername); Set googleGroups = Collections.emptySet(); if (StringUtils.isNotEmpty(googleUsername)) { + log.info("Retrieving Google groups for {}", googleUsername); googleGroups = dienst2Repository.getGoogleGroups(person.getId()); }