Skip to content
Snippets Groups Projects
Commit a7b08b3b authored by OZGCloud's avatar OZGCloud
Browse files

OZG-3961 remove logging

parent f00bfe4f
No related branches found
No related tags found
No related merge requests found
...@@ -25,7 +25,6 @@ package de.ozgcloud.operator.keycloak.user; ...@@ -25,7 +25,6 @@ package de.ozgcloud.operator.keycloak.user;
import java.util.Arrays; import java.util.Arrays;
import java.util.Optional; import java.util.Optional;
import java.util.logging.Level;
import org.keycloak.admin.client.CreatedResponseUtil; import org.keycloak.admin.client.CreatedResponseUtil;
import org.keycloak.admin.client.Keycloak; import org.keycloak.admin.client.Keycloak;
...@@ -36,16 +35,11 @@ import org.keycloak.representations.idm.UserRepresentation; ...@@ -36,16 +35,11 @@ import org.keycloak.representations.idm.UserRepresentation;
import org.springframework.beans.factory.annotation.Autowired; import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Component; import org.springframework.stereotype.Component;
import com.fasterxml.jackson.core.JsonProcessingException;
import com.fasterxml.jackson.databind.ObjectMapper;
import de.ozgcloud.operator.keycloak.KeycloakException; import de.ozgcloud.operator.keycloak.KeycloakException;
import de.ozgcloud.operator.keycloak.KeycloakGenericRemoteService; import de.ozgcloud.operator.keycloak.KeycloakGenericRemoteService;
import de.ozgcloud.operator.keycloak.KeycloakResultParser; import de.ozgcloud.operator.keycloak.KeycloakResultParser;
import lombok.extern.java.Log;
@Component @Component
@Log
class KeycloakUserRemoteService { class KeycloakUserRemoteService {
@Autowired @Autowired
...@@ -76,12 +70,6 @@ class KeycloakUserRemoteService { ...@@ -76,12 +70,6 @@ class KeycloakUserRemoteService {
public void updateUser(UserRepresentation user, String namespace) { public void updateUser(UserRepresentation user, String namespace) {
var realmResource = getRealm(namespace); var realmResource = getRealm(namespace);
try {
log.info("Updating user " + new ObjectMapper().writeValueAsString(user));
} catch (JsonProcessingException e) {
// TODO Auto-generated catch block
log.log(Level.WARNING, "", e);
}
realmResource.users().get(user.getId()).update(user); realmResource.users().get(user.getId()).update(user);
addClientRoles(user.getId(), realmResource, namespace, user); addClientRoles(user.getId(), realmResource, namespace, user);
......
...@@ -29,10 +29,7 @@ import org.apache.commons.lang3.StringUtils; ...@@ -29,10 +29,7 @@ import org.apache.commons.lang3.StringUtils;
import org.springframework.beans.factory.annotation.Autowired; import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Component; import org.springframework.stereotype.Component;
import lombok.extern.java.Log;
@Component @Component
@Log
class KeycloakUserService { class KeycloakUserService {
@Autowired @Autowired
...@@ -48,13 +45,9 @@ class KeycloakUserService { ...@@ -48,13 +45,9 @@ class KeycloakUserService {
if (userHasNoPassword(userSpec, namespace)) { if (userHasNoPassword(userSpec, namespace)) {
var secret = userSecretService.getOrCreateClusterSecret(userSpec, namespace); var secret = userSecretService.getOrCreateClusterSecret(userSpec, namespace);
log.info("Setting generated password " + userSecretService.getPasswordFromSecret(secret) + " for user "
+ userSpec.getKeycloakUser().getUsername());
userSpec.getKeycloakUser().setPassword(userSecretService.getPasswordFromSecret(secret)); userSpec.getKeycloakUser().setPassword(userSecretService.getPasswordFromSecret(secret));
} }
log.info("Userpw2 " + userSpec.getKeycloakUser().getPassword());
remoteService.getUserByName(userSpec.getKeycloakUser().getUsername(), namespace) remoteService.getUserByName(userSpec.getKeycloakUser().getUsername(), namespace)
.ifPresentOrElse(existingUser -> remoteService.updateUser(userMapper.update(existingUser, userSpec), namespace), .ifPresentOrElse(existingUser -> remoteService.updateUser(userMapper.update(existingUser, userSpec), namespace),
() -> remoteService.createUser(userMapper.map(userSpec), namespace)); () -> remoteService.createUser(userMapper.map(userSpec), namespace));
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment