Grzegorz Kossakowski wrote:
Hello guys,

I noticed comment from Dominic Mitchell at his blog complaining[1] about the 
fact that we print 30
lines long exception just to inform there is no block handling favicon.ico 
browser's request.

It is caused by following snippet in DispatcherServlet:

        if (servlet == null) {
            throw new ServletException("No block for " + req.getPathInfo());
        }

Now I start to think that throwing this exception is a bad idea. It's going to 
be quite casual
situation that server is requested non-existing resources and I think we 
shouldn't do so much noise
about it. Why about just returning 404 response with message the same as it's 
now thrown within
exception?

I'll make this change within two days if nobody complains.

+1

--
Reinhard Pötz                            Managing Director, {Indoqa} GmbH
                          http://www.indoqa.com/en/people/reinhard.poetz/

Member of the Apache Software Foundation
Apache Cocoon Committer, PMC member, PMC Chair        [EMAIL PROTECTED]
_________________________________________________________________________

Reply via email to