Thanks for confirming, I may look into this although this seems unlikely at
the moment.

+1 on using something similar to the standard db_common API for queries,
the patch's prefix approach is too simplistic and forces a specific way of
storing things in Redis. It would also probably be useful to support Hashes
and so on, which I doubt the patch does (although I only skimmed the code).

On Sun, Oct 1, 2023 at 8:22 AM Wietse Venema via Postfix-devel <
postfix-devel@postfix.org> wrote:

> Christophe Kalt via Postfix-devel:
> > Hi folks,
> >
> > https://www.mail-archive.com/postfix-devel@postfix.org/msg00952.html has
> > what looks like a nearly complete patch to support redis from 2.5 years
> > ago. Wondering whether this didn't get included in a subsequent release
> > because of the TODOs, or some other reason?
>
> This port needed more time. The work log says:
>
> +Done: use standard db_common API for queries and configuration.
> +
> +TODO:
> +
> +Update comments in src/global/dict_redis.c
> +
> +Update proto/redis_table
> +
> +Add REDIS_README with build instructions.
> +
> +Handle error result and non-error response types fom redisCommand();
> +don't assume that the result will be a string.
>
> I guess that it was at at the top of the stack for a while, and then
> was taken over by other things than needed attention.
>
>         Wietse
> _______________________________________________
> Postfix-devel mailing list -- postfix-devel@postfix.org
> To unsubscribe send an email to postfix-devel-le...@postfix.org
>
_______________________________________________
Postfix-devel mailing list -- postfix-devel@postfix.org
To unsubscribe send an email to postfix-devel-le...@postfix.org

Reply via email to