Close stream when done with it

We should have been doing this from the start. Potential cause of a memory leak?
bryanhonof · May 26, 2024 · a6ade7e · a6ade7e
1 parent 80af149
commit a6ade7e
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/pkg/gui/container_logs.go b/pkg/gui/container_logs.go
@@ -114,8 +114,10 @@ func (gui *Gui) writeContainerLogs(container *commands.Container, ctx context.Co
 		Follow:     true,
 	})
 	if err != nil {
+		gui.Log.Error(err)
 		return err
 	}
+	defer readCloser.Close()
 
 	if container.DetailsLoaded() && container.Details.Config.Tty {
 		_, err = io.Copy(writer, readCloser)