82486 2005-02-18 11:41 0000 mod_log_sql configuration file is incorrect. 2005-02-20 15:12:22 0000 1 1 1 Unclassified Gentoo Linux Server unspecified All Linux RESOLVED FIXED P2 major --- 1 chtekk@gentoo.org apache-bugs@gentoo.org chtekk@gentoo.org 2005-02-18 11:41:57 0000 Hi, I've noticed some problems with the shipped mod_log_sql config file (10_mod_log_sql.conf): 1) There is no mention of the LogSQLLoginInfo directive in that file, but that directive is strictly necessary to let the module work! Please add it. (See http://www.outoforder.cc/projects/apache/mod_log_sql/docs-2.0/#Sect.ConfigReference) 2) You use "LogSQLDBParam user loguser" etc. to specify the database connection proprieties, that should be done with LogSQLLoginInfo and thus after the change in 1) these lines are unnecessary. 3) "LogSQLCreateTables on" is there two times, one time it's commented with the mention that it isn't needed, the second time it's there, but like in the first mention: it is not needed since you turn "LogSQLMassVirtualHosting on" earlier. 4) With the new 1.100 version in portage, done with the new Apache ebuild classes, I noticed that also if you use Apache2, the config files is always /usr/portage/net-www/mod_log_sql/files/10_mod_log_sql.conf, and not 10_mod_log_sql_apache2.conf like it should be, thus you get a config file with "LogSQLPreserveFile /var/log/apache/mod_slog_sql_preserve.sql", but with Apache2 it should be "LogSQLPreserveFile /var/log/apache2/mod_slog_sql_preserve.sql". Reproducible: Always Steps to Reproduce: 1. Install the package. 2. Take a look at the default configuration file. Actual Results: The mod doesn't work/doesn't work properly. Expected Results: It should be configured correctly and work. :) chtekk@gentoo.org 2005-02-18 13:52:07 0000 Ok, examining further the config files I also found a typo in the Apache1 config file, it's "LoadModule log_sql_mysql_module extramoudles/mod_log_sql_mysql.so" in the file, but it should be "LoadModule log_sql_mysql_module extramodules/mod_log_sql_mysql.so". To help I'll attach the two rewritten config files. Also in mod_log_sql-1.100.ebuild change line 20 from: APACHE2_MOD_CONF="${PV}/10_${PN}" to: APACHE2_MOD_CONF="${PV}/10_${PN}_apache2" and then also that should work. chtekk@gentoo.org 2005-02-18 13:52:58 0000 Created an attachment (id=51548) Rewritten config files. chtekk@gentoo.org 2005-02-18 14:11:23 0000 Oups, two little errors in my previous postings: 1) It's not line 20, but line 21. 2) I forgot to mention that you also need to create a config file for Apache2 in the files/1.100/ directory if you use that modify, so you'll end up with: files/10_mod_log_sql.conf files/10_mod_log_sql_apache2.conf files/1.100/10_mod_log_sql.conf files/1.100/10_mod_log_sql_apache2.conf The 10_mod_log_sql.conf and 10_mod_log_sql_apache2.conf are both the same as mine, and I tried it and it works. chtekk@gentoo.org 2005-02-19 05:26:35 0000 Ahh another oups! :) For the files/1.100/10_mod_log_sql*.conf you have to change the LoadModule directives paths from extramodules/ to modules/, then it will work perfectly fine (atm it's logging my webserver, so it does! :D ). hollow@gentoo.org 2005-02-20 15:12:22 0000 > 1) There is no mention of the LogSQLLoginInfo directive added > 3) "LogSQLCreateTables on" is there two times fixed > 4) With the new 1.100 version in portage, done with the new Apache ebuild classes, I noticed that also if you use Apache2, the config files is always /usr/portage/net-www/mod_log_sql/files/10_mod_log_sql.conf, and not 10_mod_log_sql_apache2.conf like it should be It shouldn't > thus you get a config file with "LogSQLPreserveFile /var/log/apache/mod_slog_sql_preserve.sql", but with Apache2 it should be "LogSQLPreserveFile /var/log/apache2/mod_slog_sql_preserve.sql". fixed by using relative path information in mod_log_sql.conf i reviewed and completely redid mod_log_sql.conf it's clean now and provides good default behaviour 51548 2005-02-18 13:52 0000 Rewritten config files. mod_log_sql_correct.txt text/plain MTBfbW9kX2xvZ19zcWwuY29uZjoKCjxJZkRlZmluZSBMT0dfU1FMPgoKICAgICMgTG9hZCB0aGUg bW9kdWxlIGZpcnN0CiAgICA8SWZNb2R1bGUgIW1vZF9sb2dfc3FsLmM+CiAgICAgICAgTG9hZE1v ZHVsZSBsb2dfc3FsX21vZHVsZSAgICAgICAgZXh0cmFtb2R1bGVzL21vZF9sb2dfc3FsLnNvCiAg ICAgICAgTG9hZE1vZHVsZSBsb2dfc3FsX215c3FsX21vZHVsZSAgZXh0cmFtb2R1bGVzL21vZF9s b2dfc3FsX215c3FsLnNvCiMgICAgICAgIDxJZk1vZHVsZSBtb2Rfc3NsLmM+CiMgICAgICAgICAg ICBMb2FkTW9kdWxlIGxvZ19zcWxfc3NsX21vZHVsZSAgZXh0cmFtb2R1bGVzL21vZF9sb2dfc3Fs X3NzbC5zbwojICAgICAgICA8L0lmTW9kdWxlPgoKICAgICAgICBBZGRNb2R1bGUgbW9kX2xvZ19z cWwuYwojICAgICAgICA8SWZNb2R1bGUgbW9kX3NzbC5jPgojICAgICAgICAgICAgQWRkTW9kdWxl IG1vZF9sb2dfc3FsX3NzbC5jCiMgICAgICAgIDwvSWZNb2R1bGU+CgogICAgPC9JZk1vZHVsZT4K CiAgICA8SWZNb2R1bGUgbW9kX2xvZ19zcWwuYz4KCiAgICAgICAgIyBDaGFuZ2UgdGhpcyB3aXRo IHZhbGlkIHZhbHVlcyBmb3IgeW91ciBzZXR1cCAoY3VycmVudGx5IG9ubHkgbXlzcWwgZHJpdmVy KSEKICAgICAgICAjIEZvcm1hdDogIExvZ1NRTExvZ2luSW5mbyBkcml2ZXI6Ly91c2VybmFtZVs6 cGFzc3dvcmRdQGhvc3RuYW1lWzpwb3J0XS9kYXRhYmFzZQogICAgICAgIExvZ1NRTExvZ2luSW5m byBteXNxbDovL2xvZ3VzZXI6bDBnZzNyQGxvY2FsaG9zdC9hcGFjaGVsb2dzCgogICAgICAgIExv Z1NRTERCUGFyYW0gc29ja2V0ZmlsZSAvdmFyL3J1bi9teXNxbGQvbXlzcWxkLnNvY2sKCiAgICAg ICAgTG9nU1FMTWFjaGluZUlEIG1hY2hpbmVfaWQKICAgICAgICBMb2dTUUxQcmVzZXJ2ZUZpbGUg L3Zhci9sb2cvYXBhY2hlL21vZF9zbG9nX3NxbF9wcmVzZXJ2ZS5zcWwKCiAgICAgICAgIyBMb2dT UUxNYXNzVmlydHVhbEhvc3RpbmcgYXV0b21hdGljYWxseSBzZXRzIExvZ1NRTENyZWF0ZVRhYmxl cyB0byAnT24nCiAgICAgICAgTG9nU1FMTWFzc1ZpcnR1YWxIb3N0aW5nIE9uCiAgICAgICAgIyBM b2dTUUxDcmVhdGVUYWJsZXMgT24KCiAgICAgICAgIyB3aXRoIGNvb2tpZSBhZGQgYSAiYyIKICAg ICAgICAjIHdpdGggU1NMIGFkZCAiUXF6IgogICAgICAgICMgd2l0aCBtb2RfdXNlcnRyYWNrIGFk ZCAiSSIKICAgICAgICBMb2dTUUxUcmFuc2ZlckxvZ0Zvcm1hdCBBYWJmSGhNbVBwUnJTc1R0VXV2 CiAgICAgICAgPElmTW9kdWxlIG1vZF9zc2wuYz4KICAgICAgICAgICAgTG9nU1FMVHJhbnNmZXJM b2dGb3JtYXQgQWFiZkhoTW1QcFJyU3NUdFV1dlFxegogICAgICAgIDwvSWZNb2R1bGU+CgogICAg PC9JZk1vZHVsZT4KPC9JZkRlZmluZT4KCgoKCjEwX21vZF9sb2dfc3FsX2FwYWNoZTIuY29uZjoK CjxJZkRlZmluZSBMT0dfU1FMPgoKICAgICMgTG9hZCB0aGUgbW9kdWxlIGZpcnN0CiAgICA8SWZN b2R1bGUgIW1vZF9sb2dfc3FsLmM+CiAgICAgICAgTG9hZE1vZHVsZSBsb2dfc3FsX21vZHVsZSAg ICAgICAgZXh0cmFtb2R1bGVzL21vZF9sb2dfc3FsLnNvCiAgICAgICAgTG9hZE1vZHVsZSBsb2df c3FsX215c3FsX21vZHVsZSAgZXh0cmFtb2R1bGVzL21vZF9sb2dfc3FsX215c3FsLnNvCiMgICAg ICAgIDxJZk1vZHVsZSBtb2Rfc3NsLmM+CiMgICAgICAgICAgICBMb2FkTW9kdWxlIGxvZ19zcWxf c3NsX21vZHVsZSAgZXh0cmFtb2R1bGVzL21vZF9sb2dfc3FsX3NzbC5zbwojICAgICAgICA8L0lm TW9kdWxlPgoKICAgIDwvSWZNb2R1bGU+CgogICAgPElmTW9kdWxlIG1vZF9sb2dfc3FsLmM+Cgog ICAgICAgICMgQ2hhbmdlIHRoaXMgd2l0aCB2YWxpZCB2YWx1ZXMgZm9yIHlvdXIgc2V0dXAgKGN1 cnJlbnRseSBvbmx5IG15c3FsIGRyaXZlcikhCiAgICAgICAgIyBGb3JtYXQ6ICBMb2dTUUxMb2dp bkluZm8gZHJpdmVyOi8vdXNlcm5hbWVbOnBhc3N3b3JkXUBob3N0bmFtZVs6cG9ydF0vZGF0YWJh c2UKICAgICAgICBMb2dTUUxMb2dpbkluZm8gbXlzcWw6Ly9sb2d1c2VyOmwwZ2czckBsb2NhbGhv c3QvYXBhY2hlbG9ncwoKICAgICAgICBMb2dTUUxEQlBhcmFtIHNvY2tldGZpbGUgL3Zhci9ydW4v bXlzcWxkL215c3FsZC5zb2NrCgogICAgICAgIExvZ1NRTE1hY2hpbmVJRCBtYWNoaW5lX2lkCiAg ICAgICAgTG9nU1FMUHJlc2VydmVGaWxlIC92YXIvbG9nL2FwYWNoZTIvbW9kX3Nsb2dfc3FsX3By ZXNlcnZlLnNxbAoKICAgICAgICAjIExvZ1NRTE1hc3NWaXJ0dWFsSG9zdGluZyBhdXRvbWF0aWNh bGx5IHNldHMgTG9nU1FMQ3JlYXRlVGFibGVzIHRvICdPbicKICAgICAgICBMb2dTUUxNYXNzVmly dHVhbEhvc3RpbmcgT24KICAgICAgICAjIExvZ1NRTENyZWF0ZVRhYmxlcyBPbgoKICAgICAgICAj IHdpdGggY29va2llIGFkZCBhICJjIgogICAgICAgICMgd2l0aCBTU0wgYWRkICJRcXoiCiAgICAg ICAgIyB3aXRoIG1vZF91c2VydHJhY2sgYWRkICJJIgogICAgICAgIExvZ1NRTFRyYW5zZmVyTG9n Rm9ybWF0IEFhYmZIaE1tUHBSclNzVHRVdXYKICAgICAgICA8SWZNb2R1bGUgbW9kX3NzbC5jPgog ICAgICAgICAgICBMb2dTUUxUcmFuc2ZlckxvZ0Zvcm1hdCBBYWJmSGhNbVBwUnJTc1R0VXV2UXF6 CiAgICAgICAgPC9JZk1vZHVsZT4KCiAgICA8L0lmTW9kdWxlPgo8L0lmRGVmaW5lPg==