Well the patch seems reasonable to be commited.

Could I Remy ?

On 4/15/05, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote:
> DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
> RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
> <http://issues.apache.org/bugzilla/show_bug.cgi?id=34465>.
> ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
> INSERTED IN THE BUG DATABASE.
> 
> http://issues.apache.org/bugzilla/show_bug.cgi?id=34465
> 
> ------- Additional Comments From [EMAIL PROTECTED]  2005-04-15 11:28 -------
> (In reply to comment #6)
> > Anyway, look in org.apache.jasper.servlet.JspCServletContext.
> 
> As you said, I looked at JspCServletContext#getRealPath and
> JspCServletContext#getResource. They just generates a new URL, and does not
> check the existents of the actual file.
> 
> In 5.5.7, getResourceAsStream was used insted of getResource, and the
> implementation of getResourceAsStream is as below.
> 
>     public InputStream getResourceAsStream(String path) {
> 
>         try {
>             return (getResource(path).openStream());
>         } catch (Throwable t) {
>             return (null);
>         }
> 
>     }
> 
> Since Throwable is caught, FileNotFoundException will not be raised to
> processWebDotXml.
> 
> Well the easiest way to fix this problem, is modifiy
>             is = uri.openStream();
> to
>             try {
>                 is = uri.openStream();
>             } catch (java.io.FileNotFoundException ex) {
>                 return;
>             }
> or maybe catch a Throwable, as it does in getResourceAsStream.
> 
> I modified JspConfig.java on my local machine as above, and it correctly 
> worked
> as I wanted.
> 
> --
> Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
> ------- You are receiving this mail because: -------
> You are the assignee for the bug, or are watching the assignee.
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to