On Fri, Aug 12, 2022 at 03:16:36PM +0200, Theo Buehler wrote:
> 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()).
This is where this was copied from. Will change both.
--
:wq Claudio