diff options
| author | Paul Holzinger <pholzing@redhat.com> | 2021-06-15 11:05:00 +0200 |
|---|---|---|
| committer | Matthew Heon <mheon@redhat.com> | 2021-06-24 13:48:48 -0400 |
| commit | 6ba9617be69fe5c525b62db0af55d72205f8e5a1 (patch) | |
| tree | 7a380d425aaa4c24d467faed0a879a9529d304b6 /pkg/api/handlers/compat/exec.go | |
| parent | eb6d4b0cfe385a1d1f2b8cd2b90acb3ae2145ef7 (diff) | |
| download | podman-6ba9617be69fe5c525b62db0af55d72205f8e5a1.tar.gz podman-6ba9617be69fe5c525b62db0af55d72205f8e5a1.tar.bz2 podman-6ba9617be69fe5c525b62db0af55d72205f8e5a1.zip | |
Fix resize race with podman exec -it
When starting a process with `podman exec -it` the terminal is resized
after the process is started. To fix this allow exec start to accept the
terminal height and width as parameter and let it resize right before
the process is started.
Fixes #10560
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Diffstat (limited to 'pkg/api/handlers/compat/exec.go')
| -rw-r--r-- | pkg/api/handlers/compat/exec.go | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/pkg/api/handlers/compat/exec.go b/pkg/api/handlers/compat/exec.go index 1b7b884e0..77e62c112 100644 --- a/pkg/api/handlers/compat/exec.go +++ b/pkg/api/handlers/compat/exec.go @@ -178,8 +178,16 @@ func ExecStartHandler(w http.ResponseWriter, r *http.Request) { logrus.Error(errors.Wrapf(e, "error attaching to container %s exec session %s", sessionCtr.ID(), sessionID)) } + var size *define.TerminalSize + if bodyParams.Tty && (bodyParams.Height > 0 || bodyParams.Width > 0) { + size = &define.TerminalSize{ + Height: bodyParams.Height, + Width: bodyParams.Width, + } + } + hijackChan := make(chan bool, 1) - err = sessionCtr.ExecHTTPStartAndAttach(sessionID, r, w, nil, nil, nil, hijackChan) + err = sessionCtr.ExecHTTPStartAndAttach(sessionID, r, w, nil, nil, nil, hijackChan, size) if <-hijackChan { // If connection was Hijacked, we have to signal it's being closed |