From 9b45d50b9ba3dc244e748481ceb3c62174a2a8a2 Mon Sep 17 00:00:00 2001 From: Chris Beams Date: Mon, 13 Jun 2011 08:31:15 +0000 Subject: [PATCH] Add missing isDebugEnabled guard in FrameworkServlet Issue: SPR-8085 --- .../web/servlet/FrameworkServlet.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/FrameworkServlet.java b/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/FrameworkServlet.java index 1de5eb49006..8491f1a96db 100644 --- a/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/FrameworkServlet.java +++ b/org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/FrameworkServlet.java @@ -874,11 +874,13 @@ public abstract class FrameworkServlet extends HttpServletBean { logger.trace("Cleared thread-bound request context: " + request); } - if (failureCause != null) { - this.logger.debug("Could not complete request", failureCause); - } - else { - this.logger.debug("Successfully completed request"); + if (logger.isDebugEnabled()) { + if (failureCause != null) { + this.logger.debug("Could not complete request", failureCause); + } + else { + this.logger.debug("Successfully completed request"); + } } if (this.publishEvents) { // Whether or not we succeeded, publish an event.