[jbossws-commits] JBossWS SVN: r4871 - stack/metro/branches/hbraun/src/main/java/org/jboss/wsf/stack/sunri.

[jbossws-commits at lists.jboss.org]

Author: heiko.braun at jboss.com
Date: 2007-10-24 04:08:21 -0400 (Wed, 24 Oct 2007)
New Revision: 4871

Modified:
   stack/metro/branches/hbraun/src/main/java/org/jboss/wsf/stack/sunri/RequestHandlerImpl.java
Log:
Improved error handling and logging

Modified: stack/metro/branches/hbraun/src/main/java/org/jboss/wsf/stack/sunri/RequestHandlerImpl.java
===================================================================
--- stack/metro/branches/hbraun/src/main/java/org/jboss/wsf/stack/sunri/RequestHandlerImpl.java	2007-10-24 07:38:33 UTC (rev 4870)
+++ stack/metro/branches/hbraun/src/main/java/org/jboss/wsf/stack/sunri/RequestHandlerImpl.java	2007-10-24 08:08:21 UTC (rev 4871)
@@ -24,7 +24,6 @@
 //$Id$
 
 import com.sun.xml.ws.transport.http.servlet.ServletAdapter;
-import com.sun.xml.ws.util.exception.JAXWSExceptionBase;
 import org.jboss.logging.Logger;
 import org.jboss.wsf.spi.deployment.Endpoint;
 import org.jboss.wsf.spi.invocation.InvocationContext;
@@ -39,7 +38,6 @@
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.io.Writer;
-import java.util.logging.Level;
 
 /**
  * A request handler
@@ -52,10 +50,6 @@
    // provide logging
    private static final Logger log = Logger.getLogger(RequestHandlerImpl.class);
 
-   // TODO: unify logging
-   private static final java.util.logging.Logger logger =
-     java.util.logging.Logger.getLogger( com.sun.xml.ws.util.Constants.LoggingDomain + ".jboss.requestHandler");
-
    RequestHandlerImpl()
    {
    }
@@ -106,7 +100,7 @@
       }
       catch (Exception e)
       {
-         logger.log(Level.SEVERE, e.getMessage(), e);
+         log.error("Failed to process GET request", e);
          throw new ServletException(e.getMessage());
       }
    }
@@ -127,14 +121,9 @@
       {
          target.handle(context, req, res);
       }
-      catch (JAXWSExceptionBase e)
-      {
-         logger.log(Level.SEVERE, e.getMessage());
-         res.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR);
-      }
       catch (Throwable e)
       {
-         logger.log(Level.SEVERE, "caught throwable", e);
+         log.error("Failed to process POST request", e);
          res.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR);
       }
    }