[dev] patch for SQL logging

[Rob Lineweaver]

Greetings,

The attached patch improves the commenting and adds an example in horde/
config/conf.php for configuring SQL logging.  I had a tough time getting SQL 
logging working and found the current comments misleading.  Also attached is 
a log.sql file intended to go in horde/scripts/db/.

Currently conf.php implies that you should configure log params for DB by 
supplying username, password, and presumably host & database name parameters 
in $conf['log']['params'].  However, this method did not work for me with 
PEAR::Log and actually caused apache to segfault on my system.  The reason 
is that configuring those parameters makes PEAR::Log assume that a DB 
connection is already open (see pear/Log/sql.php).  What did work for me was 
specifying the DB params as a DSN instead, as the patch indicates.

Thanks,

Rob Lineweaver
Network Administrator
Harrisonburg City Public Schools

-- 
Linux: the penguin that flies.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: log_sql.patch
Type: application/octet-stream
Size: 563 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20030711/db3715b6/log_sql.obj
-------------- next part --------------
A non-text attachment was scrubbed...
Name: log.sql
Type: application/octet-stream
Size: 248 bytes
Desc: not available
Url : http://lists.horde.org/archives/dev/attachments/20030711/db3715b6/log.obj