You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by "Peter M. Goldstein" <pe...@yahoo.com> on 2002/07/28 00:06:51 UTC

[PATCH] Bug #6812


All,

The attached files are a patch for bug #6812, improper handling of
absolute file URLs.  Basically they consist of a check to see whether
the URL starts with '/' after the initial file:// protocol prefix.  If
so, the initial '/' is stripped off and the resulting string is taken as
the file path.  If not, the base directory obtained from the block
context is prepended to the string to obtain the file path.

--Peter 

[PATCH] Bug #6812 (take 2)

Posted by "Peter M. Goldstein" <pe...@yahoo.com>.
All,

Sorry.  I inadvertently introduced a bug in the absolute URL handling in
the previous revision of this fix by stripping off the leading '/'.
That'll cause a problem on non-Windows systems.  Here's a corrected fix.

--Peter

-----Original Message-----
From: Peter M. Goldstein [mailto:peter_m_goldstein@yahoo.com] 
Sent: Saturday, July 27, 2002 3:07 PM
To: 'James Developers List'
Subject: [PATCH] Bug #6812



All,

The attached files are a patch for bug #6812, improper handling of
absolute file URLs.  Basically they consist of a check to see whether
the URL starts with '/' after the initial file:// protocol prefix.  If
so, the initial '/' is stripped off and the resulting string is taken as
the file path.  If not, the base directory obtained from the block
context is prepended to the string to obtain the file path.

--Peter 

RE: [PATCH] Bug #6812

Posted by Danny Angus <da...@apache.org>.
What so a *nix path has to be file:////usr/?
I think file:///usr is what we want.


> -----Original Message-----
> From: Peter M. Goldstein [mailto:peter_m_goldstein@yahoo.com]
> Sent: 27 July 2002 23:07
> To: 'James Developers List'
> Subject: [PATCH] Bug #6812
> 
> 
> 
> 
> All,
> 
> The attached files are a patch for bug #6812, improper handling of
> absolute file URLs.  Basically they consist of a check to see whether
> the URL starts with '/' after the initial file:// protocol prefix.  If
> so, the initial '/' is stripped off and the resulting string is taken as
> the file path.  If not, the base directory obtained from the block
> context is prepended to the string to obtain the file path.
> 
> --Peter 
> 

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>