On Fri, Aug 12, 2022 at 12:53:59PM +0200, Claudio Jeker wrote:
> After all this slowcgi timeout fixing, here a similar thing for bgplgd(8).
> If the timeout fires, kill the bgpctl command first with SIGTERM then with
> SIGKILL. If the kill fails, close and cleanup the connection.
> I also reduced the timeout to a more reasonable 30sec. Lets see how that
> goes. This will abort full show rib commands but nobody should use those
> anyway.
Reads fine, ok
> + if (kill(c->command_pid, sig) == -1) {
> + lwarn("kill child %i after timeout", c->command_pid);
party line is to prefer %d over %i (there's another one in the
slowcgi_sig_handler()).
