Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: [joist-cvs] CVS update: joist/java/org/joist/security UnsecureServlet.java

Project highlights: Architectural Overview

joist
Discussion topic

Back to topic list

Re: [joist-cvs] CVS update: joist/java/org/joist/security UnsecureServlet.java

Author David Pellegrini <davidp at collab dot net>
Full name David Pellegrini <davidp at collab dot net>
Date 2000-09-04 00:34:36 PDT
Message Hi Daniel!

I'm glad to see someone's thinking about this. :^)

I've been wanting to address the deprecated:
    servletContext.getSe​rvlet(servletName);
for quite awhile. Your proposal looks basically sound to me. Go for it.

-davidp

dlr at tigris dot org wrote:
>
> User: dlr
> Date: 00/08/30 12:00:26
>
> Modified: java/org/joist/security UnsecureServlet.java
> Log:
> Steps for removal of getServlet() to move towards Servlet API 2.2 compliance.
>
> Revision Changes Path
> 1.37 +12 -1 joist/java/org/joist​/security/UnsecureSe​rvlet.java
>
> Index: UnsecureServlet.java
> ====================​====================​====================​=======
> RCS file: /cvs/joist/java/org/​joist/security/Unsec​ureServlet.java,v
> retrieving revision 1.36
> retrieving revision 1.37
> diff -u -r1.36 -r1.37
> --- UnsecureServlet.java 2000/08/29 05:31:22 1.36
> +++ UnsecureServlet.java 2000/08/30 19:00:25 1.37
> @@ -70,7 +70,7 @@
> * loading to WebMacro servlets.
> *
> * @author David C. Pellegrini <A HREF="mailto:davidp@​dataweb-systems.com"​>davidp@dataweb-s​ystems.com</A>​
> - * @version $Revision: 1.36 $
> + * @version $Revision: 1.37 $
> *
> * Copyright (c) 1999 dataweb systems, inc.
> * Copyright (c) 1999, 2000 Collab.Net, Inc.
> @@ -208,6 +208,17 @@
> {
> redirect = (UnsecureServlet)
> servletContext.getSe​rvlet(servletName);
> + /* TODO: Jon's list for upgrading Joist for Servlet 2.2 API
> + // Create the servlet.
> + redirect = (UnsecureServlet)
> + Class.forName(servle​tName).newInstance()​;
> +
> + // Call start()
> +
> + // Cache object in a Map, so that it's known that it's
> + // already been initialized.
> +
> + */
> }
> catch (ServletException se)
> {
>
>
>

--
-=-=-=-=-=-=-=-=-=-=​-=-=-=-=-=-=-=-=-=-=​-=-=-=-=-=-=-=-=-=-=​-=-=-=-=-=-=-=
collab.net | open source | do what's right | now hiring smart people

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

Messages

Show all messages in topic

Re: [joist-cvs] CVS update: joist/java/org/joist/security UnsecureServlet.java David Pellegrini <davidp at collab dot net> David Pellegrini <davidp at collab dot net> 2000-09-04 00:34:36 PDT
Messages per page: