More radrelay && radsqlrelay

[Nicolas Baradakis]

Alan DeKok wrote:

> > If people would be interested, it can be added to the CVS tree (pending
> > Alan's approval).
> 
>   A few comments:
> 
>   Don't use RADACCTDIR "/sql-relay" in the configuration section, use
> "${radacctdir}/sql-relay", which is a little friendlier.

Ok.

>   Also, call radius_xlat on inst->path, it makes the filename
> configurable, too.

Ok.

>   I don't think that the per-accounting type queries are necessary any
> more, at least in the CVS head.  You chould be able to save the
> CONF_SECTION in the module instance struct, and call cf_pair_find(),
> keying off of Acct-Status-Type.

Ok. (the code example you posted in the other mail helped me a lot)

>   And since it's no longer relaying, it shouldn't be called sqlrelay.
> Maybe "sql_query_log", or something like that...

I agree. What about "rlm_sql_log" then?

New version available there:

	     http://nbk.perso.cegetel.net/sql_log.tar.gz

-- 
Nicolas Baradakis