More radrelay && radsqlrelay

Nicolas Baradakis nbk at
Fri May 27 19:37:06 CEST 2005

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.


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


>   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:

Nicolas Baradakis

More information about the Freeradius-Devel mailing list