Re: cvs commit: httpd-2.0/docs/error/include bottom.html

[Justin Erenkrantz <je...@apache.org>]

On Sat, Jun 15, 2002 at 07:01:25AM -0000, rbb@apache.org wrote:
> rbb         2002/06/15 00:01:25
> 
>   Modified:    docs/error/include bottom.html
>   Log:
>   Comment out the SERVER_STRING variable from our default error documents.
>   Some people do not like having this information in their error pages, and
>   it makes sense to not do it by default.  If users want this back, they
>   can uncomment it.
>   
>   PR:	9319
>   
>   Revision  Changes    Path
>   1.7       +3 -0      httpd-2.0/docs/error/include/bottom.html
>   
>   Index: bottom.html
>   ===================================================================
>   RCS file: /home/cvs/httpd-2.0/docs/error/include/bottom.html,v
>   retrieving revision 1.6
>   retrieving revision 1.7
>   diff -u -r1.6 -r1.7
>   --- bottom.html	21 Aug 2001 01:31:23 -0000	1.6
>   +++ bottom.html	15 Jun 2002 07:01:25 -0000	1.7
>   @@ -11,7 +11,10 @@
>      <!--#config timefmt="%c" -->
>      <small><!--#echo encoding="none" var="DATE_LOCAL" --></small>
>      <BR>
>   +  <!-- Add this back in if you want to have the Server's version show in
>   +       all error documents.
>      <small><!--#echo encoding="none" var="SERVER_SOFTWARE" --></small>
>   +  -->

Um, won't mod_include still expand this?  Yes, it won't be shown, but
if the admin doesn't want it, it'd still be present in the returned
content.  They'd be better of not having that line in there at all.
And, doesn't SERVER_SOFTWARE just indicate what is already given in
Server header via ServerTokens?  Proper usage of that directive would
seem a better way to control this.  -- justin