latest CVS doesnt like nas_table no more....

[Arran Cudbard-Bell]

Alan Dekok wrote:
> A.L.M.Buxey at lboro.ac.uk wrote:
> 
>> noted another issue with latest CVS version.  after upgrading
>> as per normal to todays code i find that my SQL system doesnt
>> understand what ${nas_table} is anymore...
> 
>   I don't use SQL, but it seems to work for me.
> 
>   Can you post something that breaks but *isn't* an SQL config?
> 
>   Alan DeKok.
> --
>   http://deployingradius.com       - The web site of the book
>   http://deployingradius.com/blog/ - The blog
> - 
> List info/subscribe/unsubscribe? See http://www.freeradius.org/list/devel.html
Dammit,

secret = ${jrs_config.secret0}

Appears correct

secret = "the value of jrs_config.secret0"

Anyway the error I get with SQL

authorize_check_query = "SELECT id, UserName, Attribute, Value, op FROM 
${authcheck_table} WHERE UserName = '%{SQL-User-Name}'

WARNING: Unknown module "authcheck_table" in string expansion 
"%{authcheck_table} .

But this only happens when the query is actually executed ...

And it shouldn't really ...
The configuration variables should be substituted when the config file 
is being parsed ... not when the query is being executed..

So that should be, all run time configuration variable substitutions are 
broken and the SQL module happens to be the only one that seems to do 
them ?

-- 
Arran Cudbard-Bell (A.Cudbard-Bell at sussex.ac.uk)
Authentication, Authorisation and Accounting Officer
Infrastructure Services | ENG1 E1-1-08
University Of Sussex, Brighton
EXT:01273 873900 | INT: 3900