Re: Funny table names patch

Thomas Goirand <thomas-/[email protected]> Sat, 03 May 2008 17:15:50 +0800
Newsgroups gmane.comp.apache.mod-log-sql
Organization GPLHost
Message-ID <[email protected]>
Thomas Goirand wrote:
> Hi,
> 
> There was some bug reported in the Unbuntu's launchpad, so I had to do
> the attached patch. This escapes the table names with `` before the SQL
> queries. Please include that in the next version of mod_log_sql.

Hi,

My patch was in fact a bad idea, it's only compatible with the MySQL
backend, somebody sent some bug reports in the Debian BTS. So we'd have
to make something specific for MySQL, and that makes all together a much
more complicated patch. I'll think about it, in the mean while, do not
use it.

More generally speaking, mod_log_sql builds queries with the general
code (eg: mod_log_sql.c) when I think this should be done in the drivers
maybe. Your thoughts?

Thomas

_______________________________________________
Download the latest version at http://www.outoforder.cc/projects/apache/mod_log_sql/

To unsubscribe send an e-mail to 
mod_log_sql-unsubscribe-7qY7E20V6GW73k+5HYS8LQqVMODqnSLI@public.gmane.org