diff --git a/provisioning/ucf-impl-connid/src/main/java/com/evolveum/midpoint/provisioning/ucf/impl/connid/ConnIdUtil.java b/provisioning/ucf-impl-connid/src/main/java/com/evolveum/midpoint/provisioning/ucf/impl/connid/ConnIdUtil.java index a9c811985c2..4ebe790b170 100644 --- a/provisioning/ucf-impl-connid/src/main/java/com/evolveum/midpoint/provisioning/ucf/impl/connid/ConnIdUtil.java +++ b/provisioning/ucf-impl-connid/src/main/java/com/evolveum/midpoint/provisioning/ucf/impl/connid/ConnIdUtil.java @@ -132,8 +132,10 @@ static Throwable processConnIdException(Throwable connIdException, String desc, throw new IllegalArgumentException("Null exception while processing ConnId exception "); } - LOGGER.error("ConnId Exception {} in {}: {}", connIdException.getClass().getName(), - desc, connIdException.getMessage(), connIdException); + // We intentionally do not use LOGGER.error here, to avoid dumping the stack unless the DEBUG logging is set. + // This is because ConnID errors often get handled in upper layers. See also MID-5937. + LoggingUtils.logExceptionAsWarning(LOGGER, "Got ConnId exception (might be handled by upper layers later) {} in {}: {}", + connIdException, connIdException.getClass().getName(), desc, connIdException.getMessage()); if (connIdException instanceof RemoteWrappedException) { // brutal hack, for now