Re: rlm_rediswho with spaces

Brian Candler wrote:

So that's the first problem to fix. In rlm_rediswho.c I suspect this line...

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

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

Fixed && pushed.

I think what's needed is an inverse of redis_escape when building the command and args, which should be pretty straightforward.

I'm not sure that's a good idea. Alan DeKok.