RFC: [PATCH] vpnc-script: split disconnect and destroy

[David Woodhouse]

On Thu, 2014-02-20 at 23:31 +0800, Antonio Borneo wrote:
> 
> Just to be sure.
> You mean that in new scripts:
> - reason disconnect plus separate variable == alpha, means disconnect only;
> - reason disconnect plus separate variable == beta, means destroy only.

I think I'd use a new reason=destroy for the latter.

> Old script will receive twice call to disconnect (since ignores the
> new variable) and we have to verify this double call does not messup
> the network.

The old script would receive one call to 'disconnect', which would both
disconnect and destroy. And then a call to 'destroy', which would do
nothing.

I concede this will still lead to a complaint from the script, and thus
to random questions on silly web fora, which I held up as a disadvantage
of your approach. But I still prefer it, because it's still
fire-and-forget from the POV of the VPN client.

-- 
dwmw2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 5745 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/openconnect-devel/attachments/20140220/600bbcbf/attachment.bin>