From c53503b52b6fc6de37b6053719521054003fa50b Mon Sep 17 00:00:00 2001 From: James O'Doherty Date: Thu, 4 Jun 2026 00:21:56 -0400 Subject: clean up debugging prints and silence successful execution output - Remove leftover DEBUG prints from CLI and wireguard internal packages. - Silence stdout during successful command wrapping to ensure only the wrapped command's output is visible. - Redirect all warnings and internal errors to stderr. - Implement a verbose mode via `WG_WRAP_VERBOSE=1` to enable tunnel status messages. - Update E2E tests to use verbose mode for verification of tunnel lifecycle events. - Fix errcheck linting issue in wireguard.go and apply go fmt. --- internal/namespace/lifecycle_test.go | 6 +++--- internal/namespace/pinning.go | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'internal/namespace') diff --git a/internal/namespace/lifecycle_test.go b/internal/namespace/lifecycle_test.go index 9962e14..3e01ea9 100644 --- a/internal/namespace/lifecycle_test.go +++ b/internal/namespace/lifecycle_test.go @@ -87,7 +87,7 @@ func TestLifecycleReferenceCounting(t *testing.T) { if err := RegisterProcess(pm, profile); err != nil { t.Fatal(err) } - + // Simulate the application flow: Unregister before checking if we are the last one if err := UnregisterProcess(pm, profile); err != nil { t.Fatal(err) @@ -102,12 +102,12 @@ func TestLifecycleReferenceCounting(t *testing.T) { if err := os.WriteFile(filepath.Join(pidsDir, "1234567"), []byte(""), 0644); err != nil { t.Fatal(err) } - + // Register a real process so that pruning has something to do if err := RegisterProcess(pm, profile); err != nil { t.Fatal(err) } - + // Prune the stale one if err := PruneStalePids(pm, profile); err != nil { t.Fatal(err) diff --git a/internal/namespace/pinning.go b/internal/namespace/pinning.go index 9bf4fee..00f7c9b 100644 --- a/internal/namespace/pinning.go +++ b/internal/namespace/pinning.go @@ -72,7 +72,7 @@ func UnpinNamespace(pm *paths.PathManager, profile string) error { } // 3. Unmount and clean up blocking services. - // Since the block files are located within the profile directory, + // Since the block files are located within the profile directory, // we must unmount them before we can remove the directory. for _, p := range GetBlockPaths() { _ = unix.Unmount(p, unix.MNT_DETACH) -- cgit v1.2.3