> -----Original Message-----
> From: Ian Bicking [mailto:[EMAIL PROTECTED]]
> Sent: Wednesday, January 15, 2003 1:08 PM
> To: Webware devel
> Subject: [Webware-devel] ExtraURLPath
> 
> 
> I'm going to review the New algorithm, as used by ExtraURLPath.  These
> are the things I want to do:
> 
> * Handle the infinite loop on a 404
> * Fallback to Main.py/index.py like 0.7 behavior
> * Handle ExtraURLPath=0 (i.e., signal 404) -- this won't be triggered
>   in the current version, but hopefully after 0.8 we can 
> ditch all the 
>   old path handling code.
> 
> Is there anything else I need to look at?  I'm just going to be
> reviewing that entire piece of code, so if there's any issues I should
> be able to look at them.
> 
> I'm still not clear on the includeURL stuff -- is there 
> anything I need
> to look at with the path handling?  It didn't seem like it from my
> current understanding, but I'm still confused about the plan for that.

Unless I hear much concern to the contrary, I plan to roll the includeURL
fix into CVS tonight.  I posted a patch to try and get some preliminary
review before potentially destabilizing the tree, but with public CVS access
down, that may be more hassle than it is worth now.

It would be great if you could test in that environment, or at least take a
look at it.

Check out the extraurl patch I did as well, it is an attempt to fix some of
the problems you are planning on addressing.  

The plan on the includeURL handling stuff is as follows:

        includeURL() forward() and callMethodOfServlet() take a URL
component.

        The URL is considered relative to the current servlet in the current
context. 

        /Testing/IncludeURLTest includes "Dir/IncludeURLTest2" to actually
reference "/Testing/Dir/IncludeURLTest2".

        Absolute references are absolute within the current context.

        "/Testing/Dir/IncludeURLTest2" referencing "/Main" will reference
"/Testing/Main"

        It is not currently possible to access outside of the current
context.

A second patch, not currently planned for the 0.8 release, will introduce an
optional argument where the Context can be specified.

There was also a psp:include bug related to all of this, and I need to
double-check to make sure that it works there as well.

Does this make sense and sound reasonable?

-Stuart-



-------------------------------------------------------
This SF.NET email is sponsored by: A Thawte Code Signing Certificate 
is essential in establishing user confidence by providing assurance of 
authenticity and code integrity. Download our Free Code Signing guide:
http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0028en
_______________________________________________
Webware-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/webware-devel

Reply via email to