[Mod_log_sql] Funny table names patch

Thomas Goirand thomas at goirand.fr
Sat May 3 05:15:50 EDT 2008


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




More information about the Mod_log_sql mailing list