Author: heiko.braun(a)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);
}
}