Skip to content

modules/python-modules/webhook: Remove signal handling that kills the…#5522

Merged
kovgeri01 merged 1 commit intosyslog-ng:developfrom
kovgeri01:remove_webhook_signal_handling
Oct 6, 2025
Merged

modules/python-modules/webhook: Remove signal handling that kills the…#5522
kovgeri01 merged 1 commit intosyslog-ng:developfrom
kovgeri01:remove_webhook_signal_handling

Conversation

@kovgeri01
Copy link
Contributor

This patch removes the signal handling in the webhook code.
With this code the syslog-ng process can no longer handle the SIGTERM and SIGINT signals.

For example when the following config is used:

@version: current
@include "scl.conf"
source s_webhook_source {
    file("/tmp/src.txt");
};
destination d_file {
    file("/tmp/dst.txt");
};
log {
    source(s_webhook_source);
    destination(d_file);
    flags(flow-control);
};

and syslog-ng is ran by systemd and systemctl stop syslog-ng is requested syslog-ng will ignore the SIGTERM signal all together which will cause systemd to kill the process due to the failed timeout of the shutdown, this patch fixes this

… syslog-ng process's shutdown logic in case of SIGTERM

Signed-off-by: Kovacs, Gergo Ferenc <[email protected]>
@kovgeri01 kovgeri01 merged commit d27bc40 into syslog-ng:develop Oct 6, 2025
109 checks passed
@kovgeri01 kovgeri01 deleted the remove_webhook_signal_handling branch October 6, 2025 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants