add better logs

knecht/cmd/logs.go

@@ -4,24 +4,43 @@ import (
 	"fmt"
 	"os/exec"
 	"runtime"
+	"strings"
 
 	"github.com/spf13/cobra"
 )
 
 var logsCmd = &cobra.Command{
-	Use:   "logs",
-	Short: "Open the Dozzle log viewer in the browser",
+	Use:   "logs [container]",
+	Short: "Open Dozzle in the browser, optionally deep-linking to a container",
+	Args:  cobra.MaximumNArgs(1),
 	RunE: func(cmd *cobra.Command, args []string) error {
-		cfg, _, _, err := setup()
+		cfg, client, _, err := setup()
 		if err != nil {
 			return err
 		}
 
-		// Derive logs URL from Portainer URL (same base domain)
-		// e.g. https://portainer.home.jens.pub → https://logs.home.jens.pub
-		logsURL := deriveLogsURL(cfg.Portainer.URL)
-		fmt.Printf("Opening %s\n", logsURL)
-		return openBrowser(logsURL)
+		logsBase := deriveLogsURL(cfg.Portainer.URL)
+
+		if len(args) == 0 {
+			fmt.Printf("Opening %s\n", logsBase)
+			return openBrowser(logsBase)
+		}
+
+		containerName := args[0]
+		id, err := client.FindContainer(containerName)
+		if err != nil {
+			return err
+		}
+
+		// Dozzle uses a short ID (first 12 chars)
+		shortID := id
+		if len(shortID) > 12 {
+			shortID = shortID[:12]
+		}
+
+		url := logsBase + "/container/" + shortID
+		fmt.Printf("Opening %s\n", url)
+		return openBrowser(url)
 	},
 }
 
@@ -30,24 +49,17 @@ func init() {
 }
 
 func deriveLogsURL(portainerURL string) string {
+	domain := stripScheme(portainerURL)
 	// Replace "portainer." prefix with "logs."
-	return fmt.Sprintf("https://logs.%s", domainFromURL(portainerURL))
-}
-
-func domainFromURL(u string) string {
-	// Strip scheme and find domain after first dot
-	u = stripScheme(u)
-	for i, c := range u {
-		if c == '.' {
-			return u[i+1:]
-		}
+	if idx := strings.Index(domain, "."); idx != -1 {
+		return "https://logs." + domain[idx+1:]
 	}
-	return u
+	return "https://logs." + domain
 }
 
 func stripScheme(u string) string {
 	for _, prefix := range []string{"https://", "http://"} {
-		if len(u) > len(prefix) && u[:len(prefix)] == prefix {
+		if strings.HasPrefix(u, prefix) {
 			return u[len(prefix):]
 		}
 	}