diff --git a/core/src/main/java/feign/Logger.java b/core/src/main/java/feign/Logger.java index 7b0d5fa36e..bc693e45fe 100644 --- a/core/src/main/java/feign/Logger.java +++ b/core/src/main/java/feign/Logger.java @@ -178,7 +178,9 @@ protected Response logAndRebufferResponse(String configKey, Level logLevel, Resp @Override protected void log(String configKey, String format, Object... args) { - logger.fine(String.format(methodTag(configKey) + format, args)); + if (logger.isLoggable(java.util.logging.Level.FINE)) { + logger.fine(String.format(methodTag(configKey) + format, args)); + } } /** diff --git a/slf4j/src/main/java/feign/slf4j/Slf4jLogger.java b/slf4j/src/main/java/feign/slf4j/Slf4jLogger.java index 90888c4ffb..6f3d684da7 100644 --- a/slf4j/src/main/java/feign/slf4j/Slf4jLogger.java +++ b/slf4j/src/main/java/feign/slf4j/Slf4jLogger.java @@ -68,6 +68,8 @@ protected Response logAndRebufferResponse(String configKey, Level logLevel, Resp protected void log(String configKey, String format, Object... args) { // Not using SLF4J's support for parameterized messages (even though it would be more efficient) because it would // require the incoming message formats to be SLF4J-specific. - logger.debug(String.format(methodTag(configKey) + format, args)); + if (logger.isDebugEnabled()) { + logger.debug(String.format(methodTag(configKey) + format, args)); + } } }