← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~jameinel/launchpad/lp-serve-child-hangup into lp:launchpad

 

> It turns out it is exposed, but only conditionally, and not where I
> expected it.
>
> import signal
> signal.alarm()
>
> If you prefer, I'd be happy to use that instead. Should I be catching
> SIGALRM, or should I just let it propagate to kill the child?

Since it's supposed to be an unusual case that the connection fails
half way through, I think just letting it kill the client should be
fine.  The parent should wait on them and log their exit status.

-- 
https://code.launchpad.net/~jameinel/launchpad/lp-serve-child-hangup/+merge/50055
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jameinel/launchpad/lp-serve-child-hangup into lp:launchpad.



References