[exim-dev] [Bug 1745] redis expansions taking arbitrary para…

Página superior
Eliminar este mensaje
Responder a este mensaje
Autor: admin
Fecha:  
A: exim-dev
Asunto: [exim-dev] [Bug 1745] redis expansions taking arbitrary parameters
https://bugs.exim.org/show_bug.cgi?id=1745

--- Comment #2 from Jeremy Harris <jgh146exb@???> ---
This makes me feel that the support is not ready for primetime.

The call into the C-interface library for redis (hiredis), redisCommand()
has single- and multi-argument variants. If you use, as we currently do,
the single- call and the command-string is a SET then it seems that it must
have exactly three space-sep components - and there is NO way to quote spaces
to keep one as part of a component.

If you want to set a value that embeds a space, you are recommended (and I've
not found any alternative) to use the multi-argument version of redisCommand(),
with "SET %s %s" as the initial arg and two further args for the key and value.

To go that route we'd have to special-case SET, perhaps by parsing the
lookup argument... yuck. And who knows where else it might be needed?

Someone please tell me I'm wrong here.

--
You are receiving this mail because:
You are on the CC list for the bug.