Peter Nixon wrote:
Forgive me if this has been asked before, but shouldn't the sql_log module use the same SQL queries from as the normal sql module? Why does it have it's own set of queries in radiusd.conf?
rlm_sql_log isn't really a SQL module, as it doesn't talk to a SQL server. In fact, the module just feeds the radsqlrelay script and it doesn't share any code with rlm_sql. The module has it's own section because you're not supposed to configure both rlm_sql and rlm_sql_log at the same time. However, according to your SQL clean up idea, this section could be in a separate file etc/raddb/sql/sql_log.conf instead of radiusd.conf. Alan DeKok wrote:
The other thing is that the SQL module should be updated to auto-discover more queries like rlm_sql_log, rather than having them hard-coded. This means that any new accounting status type can be supported by simply adding the queries.
The only problem with the rlm_sql_log method is that it doesn't support the accounting_???_query_alt yet. I'm unsure if we still need the "alt" queries: since MySQL 4.1 the new syntax "INSERT ... ON DUPLICATE KEY UPDATE" allows to do something similar with a single statement. -- Nicolas Baradakis