cmd/debug.go

@@ -1,5 +1,3 @@
-//go:build windows
-
 package cmd
 
 import (

cmd/set.go

@@ -1,5 +1,3 @@
-//go:build windows
-
 package cmd
 
 import (

cmd/unset.go

@@ -1,5 +1,3 @@
-//go:build windows
-
 package cmd
 
 import (

proxy/debug.go

@@ -1,11 +1,8 @@
-//go:build windows
-
 package proxy
 
 import (
 	"errors"
 	"fmt"
-	"log"
 	"os"
 	"path/filepath"
 	"strings"
@@ -34,11 +31,9 @@ func Debug(cfgPath string) {
 	fmt.Printf("\nConfig:\n")
 	fmt.Printf("%s\n\n", path)
 
-	proxy, err := ReadSystemProxy()
+	proxy, _ := ReadSystemProxy()
 	proxyServer := proxy.Server
-	if err != nil {
+	if proxyServer == "" {
-		log.Fatal(err)
-	} else if proxyServer == "" {
 		proxyServer = "[N/A]"
 	}
 
@@ -46,11 +41,7 @@ func Debug(cfgPath string) {
 	fmt.Printf("Enabled: %t\n", proxy.Enabled)
 	fmt.Printf("Server: %s\n\n", proxyServer)
 
-	configs, err := util.ReadConfigs(cfgPath)
+	configs, _ := util.ReadConfigs(cfgPath)
-	if err != nil {
-		log.Fatal(err)
-	}
-
 	for _, config := range configs {
 		configCmd := config.Name
 		// Use command instead of name, if given

proxy/set.go

@@ -1,21 +1,18 @@
-//go:build windows
-
 package proxy
 
 import (
-	"log"
-
 	"github.com/Baipyrus/ProxySwitcher/util"
 )
 
 func Set(cfgPath string) {
+	stdin, closeFunc, _ := util.ReadyCmd()
+
 	proxy, _ := ReadSystemProxy()
 	// Set system proxy, if not already
 	if !proxy.Enabled {
 		SetSystemProxy(true)
 	}
 
-	var failed bool
 	configs, _ := util.ReadConfigs(cfgPath)
 	for _, config := range configs {
 		configCmd := config.Name
@@ -25,11 +22,8 @@ func Set(cfgPath string) {
 		}
 
 		commands := generateCommands(configCmd, config.Set, proxy.Server)
-		failed = util.ExecCmds(commands)
+		util.ExecCmds(commands, stdin)
 	}
 
-	// Additional feedback on error
+	closeFunc()
-	if failed {
-		log.Printf("One or more commands failed to execute. Run command 'debug' to see more.\n")
-	}
 }

proxy/unset.go

@@ -1,21 +1,18 @@
-//go:build windows
-
 package proxy
 
 import (
-	"log"
-
 	"github.com/Baipyrus/ProxySwitcher/util"
 )
 
 func Unset(cfgPath string) {
+	stdin, closeFunc, _ := util.ReadyCmd()
+
 	proxy, _ := ReadSystemProxy()
 	// Unset system proxy, if not already
 	if proxy.Enabled {
 		SetSystemProxy(false)
 	}
 
-	var failed bool
 	configs, _ := util.ReadConfigs(cfgPath)
 	for _, config := range configs {
 		configCmd := config.Name
@@ -25,11 +22,8 @@ func Unset(cfgPath string) {
 		}
 
 		commands := generateCommands(configCmd, config.Unset, "")
-		failed = util.ExecCmds(commands)
+		util.ExecCmds(commands, stdin)
 	}
 
-	// Additional feedback on error
+	closeFunc()
-	if failed {
-		log.Printf("One or more commands failed to execute. Run command 'debug' to see more.\n")
-	}
 }

proxy/util.go

@@ -21,8 +21,6 @@ func processVars(cmd *util.Command, isVariableType bool) {
 	for _, arg := range cmd.Arguments {
 		cmd.Name = strings.Replace(cmd.Name, "$PRSW_ARG", arg, 1)
 	}
-
-	cmd.Arguments = nil
 }
 
 func injectProxy(cmd *util.Command, variant *util.Variant, proxyServer string) {

util/command.go

@@ -1,31 +1,45 @@
-//go:build windows
-
 package util
 
 import (
+	"bytes"
 	"fmt"
-	"log"
+	"io"
 	"os/exec"
 	"strings"
 )
 
-func ExecCmds(commands []*Command) bool {
+// Create a single powershell process and leave closing, input and output open
-	var failed bool
+func ReadyCmd() (*io.WriteCloser, func() error, error) {
-	for _, command := range commands {
+	cmd := exec.Command("powershell", "-NoLogo", "-NoProfile", "-Command", "-")
-		var cmdStr string = command.Name
 
-		// Combine command into single string if args are given
+	var stdout bytes.Buffer
-		if len(command.Arguments) > 0 {
+	cmd.Stdout = &stdout
-			cmdArgs := strings.Join(command.Arguments, " ")
+	cmd.Stderr = &stdout
-			cmdStr = fmt.Sprintf("%s %s", command.Name, cmdArgs)
-		}
 
-		// Try executing command in default shell
+	stdin, err := cmd.StdinPipe()
-		cmd := exec.Command("powershell", "-NoLogo", "-NoProfile", "-Command", cmdStr)
+	if err != nil {
-		if err := cmd.Run(); err != nil {
+		return nil, nil, err
-			log.Printf("Command '%s' failed!\n", cmdStr)
+	}
-			failed = true
+
-		}
+	err = cmd.Start()
+	if err != nil {
+		return nil, nil, err
+	}
+
+	return &stdin, func() error {
+		// Close stdin pipe
+		stdin.Close()
+
+		// Wait for command to flush
+		err := cmd.Wait()
+		return err
+	}, nil
+}
+
+func ExecCmds(commands []*Command, stdin *io.WriteCloser) {
+	for _, command := range commands {
+		cmdArgs := strings.Join(command.Arguments, " ")
+		cmdStr := fmt.Sprintf("%s %s", command.Name, cmdArgs)
+		fmt.Fprintln(*stdin, cmdStr)
 	}
-	return failed
 }