Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: CVS update: MODIFIED: security ...

Project highlights: Architectural Overview

joist
Discussion topic

Back to topic list

Re: CVS update: MODIFIED: security ...

Author dlr
Full name Daniel Rall
Date 2001-05-01 23:43:30 PDT
Message Actually, by the time a HandlerException has been caught, another
exception has already been caught an rethrown. Thus, most of the
stack trace has been lost when converting to a HandlerException. Yay
HandlerException, you suck! BTW, we no longer need HandlerException
in the HEAD since I removed the UnsecureServlet dependancy on
WMServlet.

edk at tigris dot org writes:

> User: edk
> Date: 01/05/01 20:45:55
>
> Modified: java/org/joist/security UnsecureServlet.java
> Log:
> add basic error reporting ...
>
> Revision Changes Path
> 1.127 +3 -1 joist/java/org/joist​/security/UnsecureSe​rvlet.java
>
> http://joist.tigris.​org/source/browse/jo​ist/java/org/joist/s​ecurity/UnsecureServ​let.java.diff?r1=1.1​26&r2=1.127
>
> (In the diff below, changes in quantity of whitespace are not shown.)
>
> Index: UnsecureServlet.java
> ====================​====================​====================​=======
> RCS file: /usr/local/tigris/da​ta/helm/cvs/reposito​ry/joist/java/org/jo​ist/security/Unsecur​eServlet.java,v
> retrieving revision 1.126
> retrieving revision 1.127
> diff -u -b -r1.126 -r1.127
> --- UnsecureServlet.java 2001/04/17 23:25:51 1.126
> +++ UnsecureServlet.java 2001/05/02 03:45:55 1.127
> @@ -82,7 +82,7 @@
> *
> * @author <a href="mailto:davidp@​dataweb-systems.com"​>David C. Pellegrini</a>
> * @author <a href="mailto:dlr@col​lab.net">Daniel Rall</a>
> - * @version $Revision: 1.126 $
> + * @version $Revision: 1.127 $
> *
> * Copyright (c) 1999 dataweb systems, inc.
> * Copyright (c) 1999-2001 CollabNet, Inc.
> @@ -268,6 +268,8 @@
> }
> catch (HandlerException e)
> {
> + // wouldn't it be nice to see the stack trace somewhere?
> + Log.error(e);
> redirectToError(context, e.getMessage());
> }

--------------------​--------------------​--------------------​---------
To unsubscribe, e-mail: dev-unsubscribe@jois​t.tigris.org
For additional commands, e-mail: dev-help at joist dot tigris dot org

« Previous message in topic | 1 of 1 | Next message in topic »

Messages

Show all messages in topic

Re: CVS update: MODIFIED: security ... dlr Daniel Rall 2001-05-01 23:43:30 PDT
Messages per page: