CHUKWA-619. Disabled trace method on Chukwa servlets. (Julio Conca vi…

…a Eric Yang)

git-svn-id: https://svn.apache.org/repos/asf/incubator/chukwa/trunk@1214465 13f79535-47bb-0310-9956-ffa450edef68

CHANGES.txt

@@ -108,6 +108,8 @@ Trunk (unreleased changes)
 
   BUG FIXES
 
+    CHUKWA-619. Disabled trace method on Chukwa servlets. (Julio Conca via Eric Yang)
+
     CHUKWA-617. Fixed test case casting and race conditions. (Eric Yang)
 
     CHUKWA-616. Updated release note and change log generation. (Eric Yang)

src/main/java/org/apache/hadoop/chukwa/datacollection/collector/servlet/CommitCheckServlet.java

@@ -68,6 +68,11 @@ public void init(ServletConfig servletConf) throws ServletException {
       throw new ServletException(e);
     }
   }
+
+  @Override
+  protected void doTrace(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { 
+    resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED); 
+  }
 
   @Override
   protected void doGet(HttpServletRequest req, HttpServletResponse resp)

src/main/java/org/apache/hadoop/chukwa/datacollection/collector/servlet/LogDisplayServlet.java

@@ -81,6 +81,11 @@ public LogDisplayServlet(Configuration c) {
   public void init(ServletConfig servletConf) throws ServletException {
     BUF_SIZE = conf.getLong(BUF_SIZE_OPT, BUF_SIZE);
   }
+
+  @Override
+  protected void doTrace(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { 
+    resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED); 
+  }
 
   private String getSID(Chunk c) {
     try { 

src/main/java/org/apache/hadoop/chukwa/datacollection/collector/servlet/ServletCollector.java

@@ -116,6 +116,11 @@ public void run() {
     }
   }
 
+  @Override
+  protected void doTrace(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { 
+    resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED); 
+  }
+
   protected void accept(HttpServletRequest req, HttpServletResponse resp)
       throws ServletException {
     numberHTTPConnection++;

src/main/java/org/apache/hadoop/chukwa/hicc/Iframe.java

@@ -31,6 +31,11 @@
 public class Iframe extends HttpServlet {
   public static final long serialVersionUID = 100L;
 
+  @Override
+  protected void doTrace(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException { 
+    resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED); 
+  }
+
   public void doGet(HttpServletRequest request, HttpServletResponse response)
       throws IOException, ServletException {
     String id;

src/main/java/org/apache/hadoop/chukwa/hicc/Workspace.java

@@ -35,6 +35,11 @@ public class Workspace extends HttpServlet {
   private String user = "admin";
   private XssFilter xf = null;
 
+  @Override  
+  protected void doTrace(HttpServletRequest req, HttpServletResponse resp) throws ServletException, IOException {
+    resp.sendError(HttpServletResponse.SC_METHOD_NOT_ALLOWED); 
+  }
+
   public void doGet(HttpServletRequest request, HttpServletResponse response)
       throws IOException, ServletException {
     xf = new XssFilter(request);