You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by xu...@apache.org on 2011/03/15 03:28:00 UTC

svn commit: r1081643 - /geronimo/server/branches/3.0-M2/plugins/tomcat/geronimo-tomcat7/src/main/java/org/apache/geronimo/tomcat/model/HostType.java

Author: xuhaihong
Date: Tue Mar 15 02:28:00 2011
New Revision: 1081643

URL: http://svn.apache.org/viewvc?rev=1081643&view=rev
Log:
GERONIMO-5863 Web access log of Tomcat is not generated (Patch from Fang ShengHao)

Modified:
    geronimo/server/branches/3.0-M2/plugins/tomcat/geronimo-tomcat7/src/main/java/org/apache/geronimo/tomcat/model/HostType.java

Modified: geronimo/server/branches/3.0-M2/plugins/tomcat/geronimo-tomcat7/src/main/java/org/apache/geronimo/tomcat/model/HostType.java
URL: http://svn.apache.org/viewvc/geronimo/server/branches/3.0-M2/plugins/tomcat/geronimo-tomcat7/src/main/java/org/apache/geronimo/tomcat/model/HostType.java?rev=1081643&r1=1081642&r2=1081643&view=diff
==============================================================================
--- geronimo/server/branches/3.0-M2/plugins/tomcat/geronimo-tomcat7/src/main/java/org/apache/geronimo/tomcat/model/HostType.java (original)
+++ geronimo/server/branches/3.0-M2/plugins/tomcat/geronimo-tomcat7/src/main/java/org/apache/geronimo/tomcat/model/HostType.java Tue Mar 15 02:28:00 2011
@@ -559,14 +559,14 @@ public class HostType {
         for (String alias : getAlias()) {
             host.addAlias(alias);
         }
+
         //valve
-        if (host instanceof Pipeline) {
-            Pipeline pipeline = (Pipeline) host;
-            for (ValveType valveType : getValve()) {
-                Valve valve = valveType.getValve(cl);
-                pipeline.addValve(valve);
-            }
+        Pipeline pipeline = host.getPipeline();
+        for (ValveType valveType : getValve()) {
+            Valve valve = valveType.getValve(cl);
+            pipeline.addValve(valve);
         }
+
         //context
         //TODO contexts
         return host;