Attempt at SIGTERM handling

miyagawa / Starman

Starman is a high-performance preforking Perl PSGI web server

http://search.cpan.org/dist/Starman

Other

287 stars 84 forks source link

Attempt at SIGTERM handling #58

Open redhotpenguin opened 11 years ago

redhotpenguin commented 11 years ago

I found that Starman doesn't handle SIGTERM signals which are sent by default from the kill executable. I've made an attempt at adding this support to Starman. I don't have an environment available to test this (we found the issue on an integration server that our Ops team manages), but took a shot at adding this support.

jeteve commented 10 years ago

Hi there, we kind of need that for one of our application. Any chance it gets merged soon?

miyagawa commented 10 years ago

Well this is a compatibility issue, and not sure what we can do about it without breaking the users who uses the TERM signals to do what it does today.

Can you step back and explain what problem you're trying to solve?

jeteve commented 10 years ago

Ha sorry, I didn't realize TERM was already handled for something else.

Ref: https://metacpan.org/pod/distribution/Starman/script/starman

Thanks for your prompt answer.