httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Roy T. Fielding" <>
Subject Re: [PATCH] for error-notes and ap_send_error_response()
Date Fri, 25 Sep 1998 05:38:21 GMT
>The error response is completely opaque to almost all users
>(the ones that see it) anyway, with or without the details.

The errors I was handling only occur when someone is trying to hack
the server or when someone is testing a new client and running into
interoperability problems that we did not anticipate.  It is the latter
crew that needs the details (and will understand them).

>> ErrorDocument is a feature that I would never recommend to users.
>Why not?  That sounds like a personal preference; is there
>more to it?

Yes and yes.  It is an expensive operation to perform for what is
already an abnormal situation. It makes some sense for 404 and 401
(where you can give some useful website-specific instructions to the
user), but people often use it just to display a goofy graphic like

where a single bad connection is turned into three (two in-line images).
Granted, we use it pretty well given the reasonable assumption that the
broken link is in the docs, but I've seen hundreds of sites that
misuse it.  That doesn't mean I don't want it in the source -- I just
wouldn't recommend it as a general solution.

>> The correct solution is to insert the text in the normal canned
>> response rather than overriding it, and use a flag on ap_log_rerror()
>> to add the error-notes rather than making it automatic.
>I'll go along with the first but not necessarily the last.
>Modules can still stuff values in there, which may show up on
>the error page and be completely stupid.

I was under the impression that the only reason to stuff values in there
was so that they would show up on the error page.  If it is being used
for general communication to an error-handling script, then I suppose
it is right to put the information in by default and only display it
where desired.


View raw message