On 01/25/2013 10:08 PM, Michael wrote:
> Suggestion...

Thanks for the suggestion. The patches now have an implementation that
does it like this:

  AuthColumnDef n, attributename, type[, formatted]
For example:
  AuthColumnDef 1, Filter-Id, reply, formatted

Formatting is only done if it is requested for the column. Also, it
works for any type, not just for GENERIC.

Thanks again for the idea.
Heikki

> I just noticed that when using GENERIC attribute name for AuthColumnDef 
> in <AuthBy SQL> which allows for a comma separated attribute list, the 
> result from the select query is not passed through format_special 
> therefore I can't use global variables.
> 
> 
> 
> --- old/Radius/AuthSQL.pm     2013-01-07 17:21:33.000000000 -0500
> +++ new/Radius/AuthSQL.pm       2013-01-25 15:08:55.000000000 -0500
> @@ -472,6 +472,7 @@ sub getAuthColumns
>          if ($attrib eq "GENERIC")
>          {
>              # Column is a list of attr=value pairs
> +           $cols[$colnr] = &Radius::Util::format_special( 
> $cols[$colnr], $p );
>              if ($type eq 'check')
>              {
>                  $user->get_check->parse($cols[$colnr]);
> 
> _______________________________________________
> radiator mailing list
> radiator@open.com.au
> http://www.open.com.au/mailman/listinfo/radiator
> 


-- 
Heikki Vatiainen <h...@open.com.au>

Radiator: the most portable, flexible and configurable RADIUS server
anywhere. SQL, proxy, DBM, files, LDAP, NIS+, password, NT, Emerald,
Platypus, Freeside, TACACS+, PAM, external, Active Directory, EAP, TLS,
TTLS, PEAP, TNC, WiMAX, RSA, Vasco, Yubikey, MOTP, HOTP, TOTP,
DIAMETER etc. Full source on Unix, Windows, MacOSX, Solaris, VMS,
NetWare etc.
_______________________________________________
radiator mailing list
radiator@open.com.au
http://www.open.com.au/mailman/listinfo/radiator

Reply via email to