Commitd7f4ac8

authored

fix: stop vpn binary on startup failure (#150)

The vpn binary doesn't exit by itself on failure to avoid messing up RPC calls, so the parent process is expected to stop it.

1 parentefb60ca commitd7f4ac8Copy full SHA for d7f4ac8

File tree

-0

lines changed

-0

lines changed

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -172,6 +172,10 @@ await _tunnelSupervisor.StartAsync(_config.TunnelBinaryPath, HandleTunnelRpcMess`
`172`	`172`	`if(reply.MsgCase!=TunnelMessage.MsgOneofCase.Start)`
`173`	`173`	`thrownewInvalidOperationException("Tunnel did not reply with a Start response");`
`174`	`174`
	`175`	`+// If the tunnel failed to start, stop the subprocess.`
	`176`	`+if(!reply.Start.Success)`
	`177`	`+await_tunnelSupervisor.StopAsync(ct);`
	`178`	`+`
`175`	`179`	`awaitBroadcastStatus(reply.Start.Success?TunnelStatus.Started:TunnelStatus.Stopped,ct);`
`176`	`180`	`returnreply.Start;`
`177`	`181`	`}`

Comments

(0)