[Issues] [mod_log_sql 0000077]: Please integrate attached patch to support configurable logging tables like in libapache-dbilogconfig-perl

[issues at outoforder.cc]

A NOTE has been added to this issue. 
====================================================================== 
http://issues.outoforder.cc/view.php?id=77 
====================================================================== 
Reported By:                korn
Assigned To:                
====================================================================== 
Project:                    mod_log_sql
Issue ID:                   77
Category:                   Feature Request
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     new
Apache Version:             unspecified 
====================================================================== 
Date Submitted:             10-17-2007 07:30 EDT
Last Modified:              10-18-2007 08:09 EDT
====================================================================== 
Summary:                    Please integrate attached patch to support
configurable logging tables like in libapache-dbilogconfig-perl
Description: 
Hi,

This patch (against 1.101) adds support for a customizable logging table,
similar to what the Apache::DBILogConfig perl module can do. We've been
using it in production for months now, and it appears to work.

Unfortunately, it isn't fully backwards compatible with the original
version (the LogSQLTransferLogFormat directive needs to be adjusted
somewhat). None of the option letters were changed, but the list of
supported letters has been expanded and it's now possible to specify what
each field is called in the SQL table.

This makes it a lot easier to migrate from an existing dbilogconfig setup
to mod_log_sql.

We are willing to "support" the patch in the sense that if changes are
needed to better integrate it with mod_log_sql, we'll see what we can do.

Should you decide to include the patch in your distribution, we'd be
grateful if you could list Allied-Visions GmbH
(http://www.allied-visions.de/) in the credits.

Best regards,

Andras

====================================================================== 

---------------------------------------------------------------------- 
 urkle - 10-18-07 08:09  
---------------------------------------------------------------------- 
Please check out SVN trunk of mod_log_sql I have some pending changes in
there for a 1.102 release that adds basically what your patch does (for
the configurability of the SQL fields) I would just need to add in the
extra *extract* functions.  Please check it out and see if it does
everything you need.

Summary of new commands

Here is the summary of the new commands.

LogSQLShowConfig logs/sqlconfig.txt
 This command will cause a file to get created (in this case
sqlconfig.txt in the apache log directory) that will contain a dump of
the mod_log_sql registered functions and field mappings. You can use
this as interim documentation for the list of TransferLogItem full names.

LogSQLRegisterItem myitem cookie mycookie myfield INT 4
  This will register a new item "myitem" using function mycookie with
argument "mycookie" (ie find cookie arg 'mycookie') and store it in the
SQL field myfield of data type INT w/ a size of 4.
  Currently the size parameter is NOT being used, but will in the future
when I update the auto-create table code to use this configuration to
create tables and ALTER them:)

LogSQLTransferLogItems +myitem
  This command says "log all the standard sql items (or what was
configured via a previous LogSQLTransferLogFormat) and ADD to it myitem.
All of the old one letter log items now have a full named version that are
migrating to this new command. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
10-17-07 07:30  korn           New Issue                                    
10-17-07 07:30  korn           File Added: dbi.v1.101.patch                    
10-17-07 07:30  korn           Apache Version            => unspecified     
10-18-07 08:09  urkle          Note Added: 0000094                          
======================================================================