roller/src/org/roller/presentation/weblog/rss RssServlet.java,1.45,1.46

Update of /cvsroot/roller/roller/src/org/roller/presentation/weblog/rss
In directory sc8-pr-cvs1:/tmp/cvs-serv378/src/org/roller/presentation/weblog/rss

Modified Files:
        RssServlet.java 
Log Message:
error checking/logging improvements

Index: RssServlet.java
===================================================================
RCS file: 
/cvsroot/roller/roller/src/org/roller/presentation/weblog/rss/RssServlet.java,v
retrieving revision 1.45
retrieving revision 1.46
diff -C2 -d -r1.45 -r1.46
*** RssServlet.java     22 Aug 2003 21:18:27 -0000      1.45
--- RssServlet.java     17 Sep 2003 02:18:34 -0000      1.46
***************
*** 6,9 ****
--- 6,10 ----
  import org.apache.velocity.context.Context;
  import org.apache.velocity.servlet.VelocityServlet;
+ import org.roller.RollerException;
  import org.roller.presentation.RollerRequest;
  import org.roller.presentation.velocity.ContextLoader;
***************
*** 39,42 ****
--- 40,65 ----
                                    HttpServletResponse response, Context ctx)
      {
+         RollerRequest rreq = null;
+         
+         try
+         {
+             rreq = 
RollerRequest.getRollerRequest(request,getServletContext());
+         }
+         catch (RollerException e)
+         {
+             // An error initializing the request is considered to be a 404
+             mLogger.debug("RollerRequest threw Exception", e);
+             try
+             {
+                 ((HttpServletResponse)response).sendError(
+                     HttpServletResponse.SC_NOT_FOUND);
+             }
+             catch (IOException e1)
+             {
+                 mLogger.debug("IOException sending error", e);
+             }
+             return null;
+         }
+         
          try
          {
***************
*** 46,51 ****
                  this, request,  response, "", true, 8192, true);
  
-             RollerRequest rreq = 
-                 RollerRequest.getRollerRequest(request,getServletContext());
              rreq.setPageContext(pageContext);
              ContextLoader.setupContext(ctx, rreq, response);
--- 69,72 ----




-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf