Fix -open to point to skaband addr when configured for the container case
We were opening on the skaband address if using -unsafe but
not doing the same in the container case.
diff --git a/cmd/sketch/main.go b/cmd/sketch/main.go
index e37c729..3d8e14b 100644
--- a/cmd/sketch/main.go
+++ b/cmd/sketch/main.go
@@ -315,7 +315,7 @@
}
}
- // Open the debug URL in the system browser if requested
+ // Open the web UI URL in the system browser if requested
if *openBrowser {
dockerimg.OpenBrowser(ctx, ps1URL)
}
diff --git a/dockerimg/dockerimg.go b/dockerimg/dockerimg.go
index bc943b8..f934c2f 100644
--- a/dockerimg/dockerimg.go
+++ b/dockerimg/dockerimg.go
@@ -278,11 +278,16 @@
slog.ErrorContext(ctx, "LaunchContainer.postContainerInitConfig", slog.String("err", err.Error()))
errCh <- appendInternalErr(err)
}
- }()
- if config.OpenBrowser {
- OpenBrowser(ctx, "http://"+localAddr)
- }
+ // We open the browser after the init config because the above waits for the web server to be serving.
+ if config.OpenBrowser {
+ if config.SkabandAddr != "" {
+ OpenBrowser(ctx, fmt.Sprintf("%s/s/%s", config.SkabandAddr, config.SessionID))
+ } else {
+ OpenBrowser(ctx, "http://"+localAddr)
+ }
+ }
+ }()
go func() {
cmd := exec.CommandContext(ctx, "docker", "attach", cntrName)