Merge pull request #1702 from mrpalide/fix/close-app-proxy-vpn-issue

fix logic of close app by sigterm
skycoin · Dec 19, 2023 · 4e5baac · 4e5baac
2 parents 8393d9f + b8680e3
commit 4e5baac
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 2 deletions.
diff --git a/cmd/skywire-cli/commands/proxy/proxy.go b/cmd/skywire-cli/commands/proxy/proxy.go
@@ -125,7 +125,6 @@ var startCmd = &cobra.Command{
 		}
 
 		ctx, cancel := cmdutil.SignalContext(context.Background(), &logrus.Logger{})
-		defer cancel()
 		go func() {
 			<-ctx.Done()
 			cancel()

diff --git a/cmd/skywire-cli/commands/vpn/vvpn.go b/cmd/skywire-cli/commands/vpn/vvpn.go
@@ -84,7 +84,6 @@ var startCmd = &cobra.Command{
 		internal.Catch(cmd.Flags(), rpcClient.StartVPNClient(pubkey))
 		internal.PrintOutput(cmd.Flags(), nil, "Starting.")
 		ctx, cancel := cmdutil.SignalContext(context.Background(), &logrus.Logger{})
-		defer cancel()
 		go func() {
 			<-ctx.Done()
 			cancel()