windwows's architecture is still shitty

main.go

@@ -47,8 +47,17 @@ func main() {
 		Repo:     envOrDefault("GITEA_REPO", defaultRepo),
 	}
 
+	if err := platformEnsureElevated(); err != nil {
+		printError(err)
+		os.Exit(1)
+	}
+
 	args := os.Args[1:]
 
+	if len(args) > 0 && args[0] == "--elevated" {
+		args = args[1:]
+	}
+
 	if len(args) == 0 {
 		runTUI(config)
 		return

platform_linux.go

@@ -13,6 +13,10 @@ func getInstallDir() (string, error) {
 	return "/opt/logofclient", nil
 }
 
+func platformEnsureElevated() error {
+	return nil
+}
+
 func platformRunPrivileged(config Config, action string) error {
 	// Bereits als root gestartet.
 	if os.Geteuid() == 0 {

platform_windows.go

@@ -6,7 +6,6 @@ import (
 	"errors"
 	"fmt"
 	"os"
-	"os/exec"
 	"path/filepath"
 	"syscall"
 	"unsafe"
@@ -15,10 +14,22 @@ import (
 var (
 	shell32         = syscall.NewLazyDLL("shell32.dll")
 	shellExecuteExW = shell32.NewProc("ShellExecuteExW")
+
+	advapi32            = syscall.NewLazyDLL("advapi32.dll")
+	openProcessToken    = advapi32.NewProc("OpenProcessToken")
+	getTokenInformation = advapi32.NewProc("GetTokenInformation")
+
+	kernel32          = syscall.NewLazyDLL("kernel32.dll")
+	closeHandle       = kernel32.NewProc("CloseHandle")
 )
 
 const (
-	seeMaskNocloseprocess = 0x00000040
+	tokenQuery          = 0x0008
+	tokenElevationClass = 20
+
+	errorCancelled = 1223
+
+	swShow = 1
 )
 
 type shellExecuteInfo struct {
@@ -39,43 +50,41 @@ type shellExecuteInfo struct {
 	hProcess       uintptr
 }
 
-func getInstallDir() (string, error) {
-	programFiles := os.Getenv("ProgramFiles")
+type tokenElevationData struct {
+	TokenIsElevated uint32
+}
 
-	if programFiles == "" {
-		return "", errors.New("ProgramFiles ist nicht gesetzt")
+func platformEnsureElevated() error {
+	// Die erhöhte Instanz wurde bereits gestartet.
+	// Sie darf sich nicht noch einmal selbst per runas starten.
+	if hasElevatedFlag() {
+		return nil
 	}
 
-	return filepath.Join(programFiles, "logofclient"), nil
-}
-
-func platformRunPrivileged(config Config, action string) error {
-	if isWindowsAdmin() {
-		return runAction(config, action)
+	if isWindowsElevated() {
+		return nil
 	}
 
-	return runElevated(action)
+	return restartElevated()
 }
 
-func isWindowsAdmin() bool {
-	// Ein einfacher und robuster Test:
-	// Versuch, in das Windows-Systemverzeichnis zu schreiben.
-	//
-	// Da wir dort natürlich nichts verändern wollen, verwenden wir
-	// stattdessen "net session". Der Befehl funktioniert nur mit
-	// Administratorrechten.
+func hasElevatedFlag() bool {
+	for _, arg := range os.Args[1:] {
+		if arg == "--elevated" {
+			return true
+		}
+	}
 
-	cmd := exec.Command("net", "session")
-	cmd.Stdout = nil
-	cmd.Stderr = nil
-
-	return cmd.Run() == nil
+	return false
 }
 
-func runElevated(action string) error {
+func restartElevated() error {
 	exe, err := os.Executable()
 	if err != nil {
-		return fmt.Errorf("Pfad des Installers konnte nicht ermittelt werden: %w", err)
+		return fmt.Errorf(
+			"Pfad des Installers konnte nicht ermittelt werden: %w",
+			err,
+		)
 	}
 
 	verb, err := syscall.UTF16PtrFromString("runas")
@@ -88,43 +97,90 @@ func runElevated(action string) error {
 		return err
 	}
 
-	parameters, err := syscall.UTF16PtrFromString(action)
+	// Die erhöhte Instanz bekommt nur das interne Flag.
+	parameters, err := syscall.UTF16PtrFromString("--elevated")
 	if err != nil {
 		return err
 	}
 
 	info := shellExecuteInfo{
 		cbSize:       uint32(unsafe.Sizeof(shellExecuteInfo{})),
-		fMask:        seeMaskNocloseprocess,
 		lpVerb:       verb,
 		lpFile:       file,
 		lpParameters: parameters,
-		nShow:        1, // SW_SHOWNORMAL
+		nShow:        swShow,
 	}
 
-	ret, _, callErr := shellExecuteExW.Call(uintptr(unsafe.Pointer(&info)))
+	ret, _, callErr := shellExecuteExW.Call(
+		uintptr(unsafe.Pointer(&info)),
+	)
 
 	if ret == 0 {
-		// ERROR_CANCELLED = 1223:
-		// Benutzer hat den UAC-Dialog abgebrochen.
-		if errno, ok := callErr.(syscall.Errno); ok && errno == 1223 {
+		if errno, ok := callErr.(syscall.Errno); ok &&
+			errno == errorCancelled {
 			fmt.Println("Administratorrechte wurden nicht erteilt.")
-			return nil
+			return errors.New("Administratorrechte erforderlich")
 		}
 
-		return fmt.Errorf("Administratorrechte konnten nicht angefordert werden: %w", callErr)
+		return fmt.Errorf(
+			"Administratorrechte konnten nicht angefordert werden: %w",
+			callErr,
+		)
 	}
 
-	// Auf die erhöhte Instanz warten.
-	if info.hProcess != 0 {
-		handle := syscall.Handle(info.hProcess)
-		defer syscall.CloseHandle(handle)
-
-		_, err := syscall.WaitForSingleObject(handle, syscall.INFINITE)
-		if err != nil {
-			return fmt.Errorf("auf erhöhte Instanz konnte nicht gewartet werden: %w", err)
-		}
-	}
+	// WICHTIG:
+	// Die ursprüngliche Instanz beendet sich sofort.
+	// Nur die erhöhte Instanz führt danach main() weiter aus.
+	os.Exit(0)
 
 	return nil
 }
+
+func isWindowsElevated() bool {
+	var token uintptr
+
+	// GetCurrentProcess() = -1
+	ret, _, _ := openProcessToken.Call(
+		uintptr(^uint(0)),
+		tokenQuery,
+		uintptr(unsafe.Pointer(&token)),
+	)
+
+	if ret == 0 || token == 0 {
+		return false
+	}
+
+	defer closeHandle.Call(token)
+
+	var elevation tokenElevationData
+	var returned uint32
+
+	ret, _, _ = getTokenInformation.Call(
+		token,
+		tokenElevationClass,
+		uintptr(unsafe.Pointer(&elevation)),
+		uintptr(unsafe.Sizeof(elevation)),
+		uintptr(unsafe.Pointer(&returned)),
+	)
+
+	if ret == 0 {
+		return false
+	}
+
+	return elevation.TokenIsElevated != 0
+}
+
+func platformRunPrivileged(config Config, action string) error {
+	// Die komplette Windows-Anwendung läuft bereits erhöht.
+	return runAction(config, action)
+}
+
+func getInstallDir() (string, error) {
+	programFiles := os.Getenv("ProgramFiles")
+
+	if programFiles == "" {
+		return "", errors.New("ProgramFiles ist nicht gesetzt")
+	}
+
+	return filepath.Join(programFiles, "logofclient"), nil
+}

shortcut_linux.go

@@ -11,7 +11,7 @@ import (
 func platformInstallIntegration(installDir string) error {
 	binaryPath := filepath.Join(
 		installDir,
-		"Logof Client",
+		"Logofclient",
 	)
 
 	linkPath := "/usr/local/bin/logofclient"
@@ -56,7 +56,7 @@ func platformInstallIntegration(installDir string) error {
 
 	desktopEntry := fmt.Sprintf(`[Desktop Entry]
 Name=Logof Client
-Comment=Logof Client
+Comment=Logofclient
 Exec=%s
 Terminal=false
 Type=Application

shortcut_windows.go

@@ -26,7 +26,7 @@ func platformInstallIntegration(installDir string) error {
 
 	shortcutPath := filepath.Join(
 		startMenuDir,
-		"Logof Client.lnk",
+		"Logofclient.lnk",
 	)
 
 	if err := os.MkdirAll(startMenuDir, 0755); err != nil {
@@ -58,7 +58,7 @@ func platformRemoveIntegration() error {
 
 	shortcutPath := filepath.Join(
 		startMenuDir,
-		"Logof Client.lnk",
+		"Logofclient.lnk",
 	)
 
 	if err := os.Remove(shortcutPath); err != nil && !os.IsNotExist(err) {
@@ -117,7 +117,7 @@ func platformCreateShortcut(
 	}
 
 	// Beschreibung.
-	if _, err := oleutil.PutProperty(shortcut, "Description", "Logof Client"); err != nil {
+	if _, err := oleutil.PutProperty(shortcut, "Description", "Logofclient"); err != nil {
 		return fmt.Errorf("Description konnte nicht gesetzt werden: %w", err)
 	}