[Jboss-svn-commits] JBL Code SVN: r1453 - trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite

[<jbo...@li...>]

Author: rem...@jb...
Date: 2005-10-26 13:03:46 -0400 (Wed, 26 Oct 2005)
New Revision: 1453

Modified:
   trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java
Log:
- Add debug messages to the valve.

Modified: trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java
===================================================================
--- trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java	2005-10-26 13:47:50 UTC (rev 1452)
+++ trunk/labs/jbossweb/src/share/classes/org/jboss/web/rewrite/RewriteValve.java	2005-10-26 17:03:46 UTC (rev 1453)
@@ -98,8 +98,14 @@
                         // Use getResource and getResourceAsStream
                         is = getClass().getClassLoader()
                             .getResourceAsStream(resourceName);
+                        if (is != null && container.getLogger().isDebugEnabled()) {
+                            container.getLogger().debug("Read configuration from CL at " + resourceName);
+                        }
                     }
                 } else {
+                    if (container.getLogger().isDebugEnabled()) {
+                        container.getLogger().debug("Read configuration from " + file.getAbsolutePath());
+                    }
                     is = new FileInputStream(file);
                 }
             } catch (Exception e) {
@@ -108,6 +114,9 @@
         }
 
         if (is == null) {
+            if (container.getLogger().isDebugEnabled()) {
+                container.getLogger().debug("No configuration found");
+            }
             // Will use management operations to configure the valve dynamically
             return;
         }
@@ -125,7 +134,18 @@
                 Object result = parse(line);
                 if (result instanceof RewriteRule) {
                     RewriteRule rule = (RewriteRule) result;
+                    if (container.getLogger().isDebugEnabled()) {
+                        container.getLogger().debug("Add rule with pattern " + rule.getPatternString()
+                                + " and substitution " + rule.getSubstitutionString());
+                    }
                     for (int i = 0; i < conditions.size(); i++) {
+                        if (container.getLogger().isDebugEnabled()) {
+                            RewriteCond cond = (RewriteCond) conditions.get(i);
+                            container.getLogger().debug("Add condition " + cond.getCondPattern() 
+                                    + " test " + cond.getTestString() + " to rule with pattern " 
+                                    + rule.getPatternString() + " and substitution " 
+                                    + rule.getSubstitutionString());
+                        }
                         rule.addCondition((RewriteCond) conditions.get(i));
                     }
                     conditions.clear();
@@ -196,6 +216,10 @@
         for (int i = 0; i < rules.length; i++) {
             CharSequence newurl = rules[i].evaluate(url, resolver);
             if (newurl != null && !url.equals(newurl.toString())) {
+                if (container.getLogger().isDebugEnabled()) {
+                    container.getLogger().debug("Rewrote " + url + " as " + newurl
+                            + " with rule pattern " + rules[i].getPatternString());
+                }
                 url = newurl.toString();
                 rewritten = true;
             }