fix: use io.Copy for exec TTY output instead of stdcopy (#4382)

robertbreker · web-flow · commit f3afbf843cc2 · 2026-01-24T16:56:48.000-08:00
diff --git a/internal/support/docker/docker_service.go b/internal/support/docker/docker_service.go
@@ -208,7 +208,8 @@ func (d *DockerClientService) Exec(ctx context.Context, c container.Container, c
 	})
 
 	wg.Go(func() {
-		if _, err := stdcopy.StdCopy(stdout, stdout, session.Reader); err != nil {
+		// TTY mode outputs raw bytes without Docker's multiplexing headers.
+		if _, err := io.Copy(stdout, session.Reader); err != nil {
 			log.Error().Err(err).Msg("error while writing to ws")
 		}
 		cancel()

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,8 @@ func (d *DockerClientService) Exec(ctx context.Context, c container.Container, c`
`208`	`208`	`})`
`209`	`209`
`210`	`210`	`wg.Go(func() {`
`211`		`- if _, err := stdcopy.StdCopy(stdout, stdout, session.Reader); err != nil {`
	`211`	`+ // TTY mode outputs raw bytes without Docker's multiplexing headers.`
	`212`	`+ if _, err := io.Copy(stdout, session.Reader); err != nil {`
`212`	`213`	`log.Error().Err(err).Msg("error while writing to ws")`
`213`	`214`	`}`
`214`	`215`	`cancel()`