Skip logging client errors

wso2-extensions · Feb 27, 2024 · 62544cc · 62544cc
1 parent 998419e
commit 62544cc
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...cim2.common/src/main/java/org/wso2/carbon/identity/scim2/common/impl/SCIMUserManager.java b/...cim2.common/src/main/java/org/wso2/carbon/identity/scim2/common/impl/SCIMUserManager.java
@@ -1129,7 +1129,6 @@ public User updateUser(User user, Map<String, Boolean> requiredAttributes) throw
             throw new BadRequestException(errorMessage, ResponseCodeConstants.INVALID_VALUE);
         } catch (UserStoreException e) {
             String errMsg = "Error while updating attributes of user: " + user.getUserName();
-            log.error(errMsg, e);
             // Sometimes client exceptions are wrapped in the super class.
             // Therefore checking for possible client exception.
             Throwable ex = ExceptionUtils.getRootCause(e);
@@ -1142,6 +1141,7 @@ public User updateUser(User user, Map<String, Boolean> requiredAttributes) throw
                 throw new BadRequestException(errorMessage, ResponseCodeConstants.INVALID_VALUE);
             }
             handleErrorsOnUserNameAndPasswordPolicy(e);
+            log.error(errMsg, e);
             if (isNotifyUserstoreStatusEnabled()) {
                 throw resolveError(e, errMsg + ". " + e.getMessage());
             } else {