You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@shiro.apache.org by lh...@apache.org on 2009/09/15 17:17:39 UTC
svn commit: r815362 -
/incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/servlet/OncePerRequestFilter.java
Author: lhazlewood
Date: Tue Sep 15 15:17:39 2009
New Revision: 815362
URL: http://svn.apache.org/viewvc?rev=815362&view=rev
Log:
Changed logging message to include the filter name for easier tracing
Modified:
incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/servlet/OncePerRequestFilter.java
Modified: incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/servlet/OncePerRequestFilter.java
URL: http://svn.apache.org/viewvc/incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/servlet/OncePerRequestFilter.java?rev=815362&r1=815361&r2=815362&view=diff
==============================================================================
--- incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/servlet/OncePerRequestFilter.java (original)
+++ incubator/shiro/trunk/web/src/main/java/org/apache/shiro/web/servlet/OncePerRequestFilter.java Tue Sep 15 15:17:39 2009
@@ -176,16 +176,12 @@
String alreadyFilteredAttributeName = getAlreadyFilteredAttributeName();
if (request.getAttribute(alreadyFilteredAttributeName) != null || shouldNotFilter(request)) {
- if (log.isTraceEnabled()) {
- log.trace("Filter already executed. Proceeding without invoking this filter.");
- }
+ log.trace("Filter '{}' already executed. Proceeding without invoking this filter.", getName());
// Proceed without invoking this filter...
filterChain.doFilter(request, response);
} else {
// Do invoke this filter...
- if (log.isTraceEnabled()) {
- log.trace("Filter not yet executed. Executing now.");
- }
+ log.trace("Filter '{}' not yet executed. Executing now.", getName());
request.setAttribute(alreadyFilteredAttributeName, Boolean.TRUE);
doFilterInternal(request, response, filterChain);
}