ASSERT FAILED

Jakob Hirsch jh at plonk.de
Wed May 15 12:59:10 CEST 2013


Alan DeKok, 13.05.2013 18:30:
>> (10) ERROR: %{md5:changeme%{tolower:%{User-Name}}%{%{Operator-Name}:-}}
> %{md5:changeme%{tolower:%{User-Name}}%{%{Operator-Name}:-""}}

I may be missing something here, but aren't unset attributes just
expanded to an empty string? If so, what is difference between the
outcome of
  %{%{Unset-Attribute}:-""}
and a simple
  %{Unset-Attribute}
?


Oh, and btw: As the :- syntax is copied from Unix shells, shouldn't it
just be "%{Attribute-Name:-attribute not set}", i.e. without additional
"%{}"? In shells it's "${VARNAME:-VARNAME is empty}", not "${${VAR}:-VAR
is empty}" (except if you have stored a variable name VAR, of course).
Using :- with a static string doesn't make much sense, so the extra %{}
is kind of redundant. But I guess it's too late for such a change now
(maybe not for 3.0).



Regards
Jakob


More information about the Freeradius-Devel mailing list