Received: (qmail 56345 invoked by uid 501); 20 Sep 2000 23:33:25 -0000
Message-Id: <20000920233325.56344.qmail@locus.apache.org>
Date: 20 Sep 2000 23:33:25 -0000
From: Mike Abbott <mja@sgi.com>
Reply-To: mja@sgi.com
To: submit@bugz.apache.org
Subject: incorrect CLF format documented
X-Send-Pr-Version: 3.110

>Number:         6556
>Category:       mod_log-any
>Synopsis:       incorrect CLF format documented
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    apache
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          doc-bug
>Submitter-Id:   apache
>Arrival-Date:   Wed Sep 20 16:40:01 PDT 2000
>Closed-Date:    Sat Sep 30 11:51:38 PDT 2000
>Last-Modified:  Sat Sep 30 11:51:38 PDT 2000
>Originator:     mja@sgi.com
>Release:        2.0a6
>Organization:
>Environment:
irrelevant
>Description:
The documentation for mod_log_config incorrectly states that the format
for Common Log Format includes %s when it actually includes %>s.  The
default format string, which happens to be CLF, is also incorrectly
documented.  The documentation was not updated at the same time as the
source code when the default format string was changed from %s to %>s to
log CLF correctly (mod_log_config.c version 1.18).  This error should
be fixed in both 1.3 and 2.0.
>How-To-Repeat:

>Fix:
--- mod_log_config.html-orig    Wed Sep 20 16:19:33 2000
+++ mod_log_config.html Wed Sep 20 16:21:03 2000
@@ -179,9 +179,9 @@
 <P>
 
 Note that the common log format is defined by the string <CODE>"%h %l
-%u %t \"%r\" %s %b"</CODE>, which can be used as the basis for
+%u %t \"%r\" %&gt;s %b"</CODE>, which can be used as the basis for
 extending for format if desired (<EM>e.g.</EM>, to add extra fields at the end).
-NCSA's extended/combined log format would be <CODE>"%h %l %u %t \"%r\" %s %b \"%{Referer}i\" \"%{User-agent}i\""</CODE>.
+NCSA's extended/combined log format would be <CODE>"%h %l %u %t \"%r\" %&gt;s %b \"%{Referer}i\" \"%{User-agent}i\""</CODE>.
 
 <P>
 
@@ -370,7 +370,7 @@
  HREF="directive-dict.html#Default"
  REL="Help"
 ><STRONG>Default:</STRONG></A> <CODE>LogFormat &quot;%h %l %u %t \&quot;%r\&quot;
-%s %b&quot;</CODE><BR>
+%&gt;s %b&quot;</CODE><BR>
 <A
  HREF="directive-dict.html#Context"
  REL="Help"
>Release-Note:
>Audit-Trail:
State-Changed-From-To: open-closed
State-Changed-By: slive
State-Changed-When: Sat Sep 30 11:51:36 PDT 2000
State-Changed-Why:
Thanks.  That's now fixed in 1.3 and 2.0.
>Unformatted:
 [In order for any reply to be added to the PR database, you need]
 [to include <apbugs@Apache.Org> in the Cc line and make sure the]
 [subject line starts with the report component and number, with ]
 [or without any 'Re:' prefixes (such as "general/1098:" or      ]
 ["Re: general/1098:").  If the subject doesn't match this       ]
 [pattern, your message will be misfiled and ignored.  The       ]
 ["apbugs" address is not added to the Cc line of messages from  ]
 [the database automatically because of the potential for mail   ]
 [loops.  If you do not include this Cc, your reply may be ig-   ]
 [nored unless you are responding to an explicit request from a  ]
 [developer.  Reply only with text; DO NOT SEND ATTACHMENTS!     ]