russdill / juniper-vpn-py

Python Juniper VPN Authenticator
GNU Lesser General Public License v2.1
103 stars 80 forks source link

Send SIGINT instead of SIGTERM to terminate child processes #5

Closed seanmil closed 9 years ago

seanmil commented 9 years ago

When calling openconnect (the typical use case for this script) it only properly cleans up the connection when sent SIGINT, not SIGTERM.

This could be made a configurable option, but I wasn't convinced that made any sense. See the discussion at http://permalink.gmane.org/gmane.network.vpn.openconnect.devel/2451 which discusses this behavior in openconnect.

seanmil commented 9 years ago

This patch has a problem. I had a couple of different fixes and sent the wrong one. Fixed version coming.