On Wed, Jul 15, 2026 at 06:14:30PM +0200, Alvaro Herrera wrote:
> Michael, looking at 
> https://github.com/michaelpq/pg_plugins/blob/main/jsonlog/jsonlog.c
> do you see an issue with this variable?

I have not directly tested it yet (will perhaps do so later this
week or later), but I am pretty sure that this module is broken as-is
with v19 under EXEC_BACKEND.  Not checking for syslogger_setup_done
and only redirection_done means that the module could try to send some
chunks of data to the syslogger without the file handles being set,
crashing the syslogger the same way as what I have done upthread.

I have created an item to look at that later, for now:
https://github.com/michaelpq/pg_plugins/issues/56

AFAIK, jsonlog is still used in some container setups even with the
in-core option in place since v15, folks have been keeping asking for
it.  I suspect that there is close to zero use of it under
EXEC_BACKEND, but we surely should fix the issue anyway..
--
Michael

Attachment: signature.asc
Description: PGP signature

Reply via email to