Re: rlm_rediswho with spaces

On 26/10/12 12:18, Brian Candler wrote:

is that the expansions of %{Tmp-String-1} and %{Tmp-String-2} are passed via redis_escape_function? That means they would expand to

Yes

if (!radius_xlat(query, sizeof (query), fmt, request, NULL)) {

...just needs the NULL changing to inst->redis_escape_func.

Probably

I think there may be a problem with that approach: given

start-insert = "LPUSH %{User-Name} %{Event-Timestamp},%{Acct-Status-Type},..."

I don't want %{Event-Timestamp} and %{Acct-Status-Type} to be treated as separate arguments.

Ah. Then yes, you'll need to parse the string before passing to xlat.