You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by ji...@apache.org on 2003/12/04 15:44:49 UTC

cvs commit: apache-1.3/src/modules/standard mod_log_config.c

jim         2003/12/04 06:44:49

  Modified:    src      CHANGES
               src/modules/standard mod_log_config.c
  Log:
  Add '%X' as an alias for '%c' in LogFormat. Both work and
  do the same thing, so we don't have backwards compatibility
  problems, but now people can adjust to use '%X' to log
  the connection status when using mod_ssl (which changes
  what '%c' means)
  
  Obtained from: Apache 2.x
  
  Revision  Changes    Path
  1.1913    +5 -0      apache-1.3/src/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/apache-1.3/src/CHANGES,v
  retrieving revision 1.1912
  retrieving revision 1.1913
  diff -u -r1.1912 -r1.1913
  --- CHANGES	17 Nov 2003 17:14:52 -0000	1.1912
  +++ CHANGES	4 Dec 2003 14:44:49 -0000	1.1913
  @@ -1,5 +1,10 @@
   Changes with Apache 1.3.30
   
  +  *) '%X' is now accepted as an alias for '%c' in the
  +     LogFormat directive. This allows you to configure logging
  +     to still log the connection status even with mod_ssl
  +     (which changes what '%c' means). [Jim Jagielski]
  +
     *) UseCanonicalName off was ignoring the client provided
        port information. [Jim Jagielski]
   
  
  
  
  1.91      +34 -30    apache-1.3/src/modules/standard/mod_log_config.c
  
  Index: mod_log_config.c
  ===================================================================
  RCS file: /home/cvs/apache-1.3/src/modules/standard/mod_log_config.c,v
  retrieving revision 1.90
  retrieving revision 1.91
  diff -u -r1.90 -r1.91
  --- mod_log_config.c	3 Feb 2003 17:13:28 -0000	1.90
  +++ mod_log_config.c	4 Dec 2003 14:44:49 -0000	1.91
  @@ -118,9 +118,11 @@
    * literal characters copied into the log files, and '%' directives as
    * follows:
    *
  - * %...B:  bytes sent, excluding HTTP headers.
  + * %...a:  remote IP-address
  + * %...A:  local IP-address
    * %...b:  bytes sent, excluding HTTP headers in CLF format, i.e. a '-'
    *         when no bytes where sent (rather than a '0'.
  + * %...B:  bytes sent, excluding HTTP headers.
    * %...c:  Status of the connection.
    *         'X' = connection aborted before the response completed.
    *         '+' = connection may be kept alive after the response is sent.
  @@ -128,15 +130,16 @@
    * %...{FOOBAR}e:  The contents of the environment variable FOOBAR
    * %...f:  filename
    * %...h:  remote host
  - * %...a:  remote IP-address
  - * %...A:  local IP-address
  + * %...H:  the request protocol
    * %...{Foobar}i:  The contents of Foobar: header line(s) in the request
    *                 sent to the client.
    * %...l:  remote logname (from identd, if supplied)
  + * %...m:  the request method
    * %...{Foobar}n:  The contents of note "Foobar" from another module.
    * %...{Foobar}o:  The contents of Foobar: header line(s) in the reply.
    * %...p:  the port the request was served to
    * %...P:  the process ID of the child that serviced the request.
  + * %...q:  the query string prepended by "?", or empty if no query string
    * %...r:  first line of request
    * %...s:  status.  For requests that got internally redirected, this
    *         is status of the *original* request --- %...>s for the last.
  @@ -148,9 +151,7 @@
    * %...U:  the URL path requested.
    * %...v:  the configured name of the server (i.e. which virtual host?)
    * %...V:  the server name according to the UseCanonicalName setting
  - * %...m:  the request method
  - * %...H:  the request protocol
  - * %...q:  the query string prepended by "?", or empty if no query string
  + * %...X:  An alias for %..c (Status of the connection).
    *
    * The '...' can be nothing at all (e.g. "%h %u %r %s %b"), or it can
    * indicate conditions for inclusion of the item (which will cause it
  @@ -500,9 +501,6 @@
       int want_orig_default;
   } log_item_keys[] = {
   
  -    {
  -        'h', log_remote_host, 0
  -    },
       {   
           'a', log_remote_address, 0 
       },
  @@ -510,70 +508,76 @@
           'A', log_local_address, 0 
       },
       {
  -        'l', log_remote_logname, 0
  +        'b', clf_log_bytes_sent, 0
       },
       {
  -        'u', log_remote_user, 0
  +        'B', log_bytes_sent, 0
       },
       {
  -        't', log_request_time, 0
  +        'c', log_connection_status, 0
       },
       {
  -        'T', log_request_duration, 1
  +        'e', log_env_var, 0
       },
       {
  -        'r', log_request_line, 1
  +        'f', log_request_file, 0
       },
       {
  -        'f', log_request_file, 0
  +        'h', log_remote_host, 0
       },
       {
  -        'U', log_request_uri, 1
  +        'H', log_request_protocol, 0
       },
       {
  -        's', log_status, 1
  +        'i', log_header_in, 0
       },
       {
  -        'b', clf_log_bytes_sent, 0
  +        'l', log_remote_logname, 0
       },
       {
  -        'B', log_bytes_sent, 0
  +        'm', log_request_method, 0
       },
       {
  -        'i', log_header_in, 0
  +        'n', log_note, 0
       },
       {
           'o', log_header_out, 0
       },
       {
  -        'n', log_note, 0
  +        'p', log_server_port, 0
       },
       {
  -        'e', log_env_var, 0
  +        'P', log_child_pid, 0
       },
       {
  -        'V', log_server_name, 0
  +        'q', log_request_query, 0
       },
       {
  -        'v', log_virtual_host, 0
  +        'r', log_request_line, 1
       },
       {
  -        'p', log_server_port, 0
  +        's', log_status, 1
       },
       {
  -        'P', log_child_pid, 0
  +        't', log_request_time, 0
       },
       {
  -        'H', log_request_protocol, 0
  +        'T', log_request_duration, 1
       },
       {
  -        'm', log_request_method, 0
  +        'u', log_remote_user, 0
       },
       {
  -        'q', log_request_query, 0
  +        'U', log_request_uri, 1
       },
       {
  -        'c', log_connection_status, 0
  +        'v', log_virtual_host, 0
  +    },
  +    {
  +        'V', log_server_name, 0
  +    },
  +    {
  +        'X', log_connection_status, 0
       },
       {
           '\0'