Hi, Another small xlat parsing error, alternate values arnt being parsed correctly in xlat strings involving modules. update request { Supplicant-Flags = "%{sql_clients:SELECT EXPORT_SET(master.supplicant_flags,'1','0','',10) FROM `master` WHERE master.hw_address = '%{Calling-Station-Id:-null}' LIMIT 0,1:-null}" } expands to SELECT EXPORT_SET(master.supplicant_flags,'1','0','',10) FROM `master` WHERE master.hw_address = '%{Calling-Station-Id:-null}' LIMIT 0,1:-null when should expand to SELECT EXPORT_SET(master.supplicant_flags,'1','0','',10) FROM `master` WHERE master.hw_address = '%{Calling-Station-Id:-null}' LIMIT 0,1 What would be really cool is if one query returned a null string a second query could be executed as an alternate, but i'm not sure how hard that would be to do. -- Arran Cudbard-Bell (A.Cudbard-Bell@sussex.ac.uk) Authentication, Authorisation and Accounting Officer Infrastructure Services | ENG1 E1-1-08 University Of Sussex, Brighton EXT:01273 873900 | INT: 3900
Arran Cudbard-Bell wrote:
alternate values arnt being parsed correctly in xlat strings involving modules.
It doesn't work. It's not intended to work, because ":-" is a perfectly valid string to pass to a module. ":-" only works for attributes.
What would be really cool is if one query returned a null string a second query could be executed as an alternate, but i'm not sure how hard that would be to do.
That's probably not hard. %{%{sql:foo}:-%{sql:bar}} It's easier to parse. It looks slightly more complicated to look at, but not too bad. Alan DeKok.
Alan DeKok wrote:
Arran Cudbard-Bell wrote:
alternate values arnt being parsed correctly in xlat strings involving modules.
It doesn't work. It's not intended to work, because ":-" is a perfectly valid string to pass to a module. ":-" only works for attributes.
Yes I suppose it is ..
What would be really cool is if one query returned a null string a second query could be executed as an alternate, but i'm not sure how hard that would be to do.
That's probably not hard.
%{%{sql:foo}:-%{sql:bar}}
It's easier to parse. It looks slightly more complicated to look at, but not too bad.
That would be excellent ... Allows If user not found here look there type stuff. Arran
participants (2)
-
Alan DeKok -
Arran Cudbard-Bell