default linelog Accounting-Request handling broken?
Alan DeKok
aland at deployingradius.com
Sun Dec 6 19:43:49 CET 2009
Josip Rodin wrote:
> Speaking of which, while browsing 558c51a711d7a326604eda0fc51040cbda2b77a8
> I just noticed that it would be good for these error messages be elevated
> so that users can see them outside of debug mode. Specifically:
>
> RDEBUG2("Badly formatted variable: %s", *from);
> RDEBUG2("Badly formatted variable: %s", p);
> RDEBUG2("No trailing :- after variable at %s", p);
> RDEBUG2("Invalid text after :- at %s", p);
>
> Those should be error log, while:
>
> RDEBUG2("Improperly nested variable; %%{%s}", p);
> RDEBUG2("WARNING: Deprecated conditional expansion \":-\". See \"man unlang\" for details");
> RDEBUG2("WARNING: Unknown module \"%s\" in string expansion \"%%%s\"", xlat_name, *from);
>
> Those looks like info messages.
Are you sure you want these messages for *every* packet? Wouldn't it
be better to warn about mis-configurations in debug mode only?
Alan DeKok.
More information about the Freeradius-Users
mailing list