You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by am...@apache.org on 2003/02/10 05:25:27 UTC

cvs commit: jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve SaveAccessLogValveAction.java SaveRemoteAddrValveAction.java SaveRemoteHostValveAction.java SaveRequestDumperValveAction.java SaveSingleSignOnValveAction.java ValveUtil.java

amyroh      2003/02/09 20:25:26

  Modified:    webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve
                        SaveAccessLogValveAction.java
                        SaveRemoteAddrValveAction.java
                        SaveRemoteHostValveAction.java
                        SaveRequestDumperValveAction.java
                        SaveSingleSignOnValveAction.java ValveUtil.java
  Log:
  Correct the behavior of valve creation - it wasn't saving its attributes during creation.
  
  Revision  Changes    Path
  1.4       +7 -7      jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveAccessLogValveAction.java
  
  Index: SaveAccessLogValveAction.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveAccessLogValveAction.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- SaveAccessLogValveAction.java	22 Nov 2002 21:49:46 -0000	1.3
  +++ SaveAccessLogValveAction.java	10 Feb 2003 04:25:26 -0000	1.4
  @@ -155,9 +155,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
           
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.4       +8 -7      jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteAddrValveAction.java
  
  Index: SaveRemoteAddrValveAction.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteAddrValveAction.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- SaveRemoteAddrValveAction.java	12 Apr 2002 22:06:30 -0000	1.3
  +++ SaveRemoteAddrValveAction.java	10 Feb 2003 04:25:26 -0000	1.4
  @@ -154,9 +154,10 @@
                  
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.4       +7 -7      jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteHostValveAction.java
  
  Index: SaveRemoteHostValveAction.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRemoteHostValveAction.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- SaveRemoteHostValveAction.java	12 Apr 2002 22:06:30 -0000	1.3
  +++ SaveRemoteHostValveAction.java	10 Feb 2003 04:25:26 -0000	1.4
  @@ -155,9 +155,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
   
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.3       +7 -7      jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRequestDumperValveAction.java
  
  Index: SaveRequestDumperValveAction.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveRequestDumperValveAction.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- SaveRequestDumperValveAction.java	12 Apr 2002 22:06:30 -0000	1.2
  +++ SaveRequestDumperValveAction.java	10 Feb 2003 04:25:26 -0000	1.3
  @@ -155,9 +155,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
           
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
              
           }
   
  
  
  
  1.5       +7 -7      jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveSingleSignOnValveAction.java
  
  Index: SaveSingleSignOnValveAction.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/SaveSingleSignOnValveAction.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- SaveSingleSignOnValveAction.java	18 Apr 2002 21:39:45 -0000	1.4
  +++ SaveSingleSignOnValveAction.java	10 Feb 2003 04:25:26 -0000	1.5
  @@ -156,9 +156,9 @@
           // Perform a "Create Valve" transaction (if requested)
           if ("Create".equals(adminAction)) {
   
  -           return ValveUtil.
  -                  createValve(parent, valveType, response, request, mapping,
  -                  (ApplicationServlet) getServlet());
  +            vObjectName = ValveUtil.createValve(parent, valveType, 
  +                                response, request, mapping, 
  +                                (ApplicationServlet) getServlet());
                         
           }
   
  
  
  
  1.8       +10 -9     jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/ValveUtil.java
  
  Index: ValveUtil.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/webapps/admin/WEB-INF/classes/org/apache/webapp/admin/valve/ValveUtil.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- ValveUtil.java	9 May 2002 01:07:31 -0000	1.7
  +++ ValveUtil.java	10 Feb 2003 04:25:26 -0000	1.8
  @@ -112,7 +112,7 @@
       
       // --------------------------------------------------------- Public Methods
       
  -    public static ActionForward createValve(String parent, String valveType,
  +    public static String createValve(String parent, String valveType,
       HttpServletResponse response, HttpServletRequest request,
       ActionMapping mapping, ApplicationServlet servlet)
       throws IOException, ServletException {
  @@ -132,6 +132,7 @@
           
           String operation = null;
           String values[] = null;
  +        String vObjectName = null;
           
           try {
               
  @@ -157,8 +158,7 @@
                   servlet.log(message);
                   response.sendError(HttpServletResponse.SC_BAD_REQUEST, message);
                   return (null);
  -            }
  -                        
  +            }    
               // Ensure that the requested valve name is unique
               
               // TBD -- do we need this check?
  @@ -187,7 +187,8 @@
               operation = "create" + valveType;
               if ("AccessLogValve".equalsIgnoreCase(valveType))
                   operation = "createAccessLoggerValve";
  -            String vObjectName = (String)
  +                
  +            vObjectName = (String)
                           mBServer.invoke(fname, operation, values, createStandardValveTypes);
               
               // Add the new Valve to our tree control node
  @@ -235,6 +236,6 @@
           
           // Forward to the success reporting page
           session.removeAttribute(mapping.getAttribute());
  -        return (mapping.findForward("Save Successful"));
  +        return vObjectName;
       }
   }
  
  
  

---------------------------------------------------------------------
To unsubscribe, e-mail: tomcat-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: tomcat-dev-help@jakarta.apache.org