summaryrefslogtreecommitdiff
path: root/pkg/api/server/register_exec.go
diff options
context:
space:
mode:
authorDaniel J Walsh <dwalsh@redhat.com>2021-03-26 10:54:33 -0400
committerDaniel J Walsh <dwalsh@redhat.com>2021-03-26 15:15:17 -0400
commitdcabf6dd717ad495ba71d2788a64255defb852fd (patch)
treebd2ee4042feadd0b84a2b4076b2d7e948f4855b7 /pkg/api/server/register_exec.go
parentc81e273835303664d62c4e958a7f34877c05ed49 (diff)
downloadpodman-dcabf6dd717ad495ba71d2788a64255defb852fd.tar.gz
podman-dcabf6dd717ad495ba71d2788a64255defb852fd.tar.bz2
podman-dcabf6dd717ad495ba71d2788a64255defb852fd.zip
Remove resize race condition
Since podman-remote resize requests can come in at random times, this generates a real potential for race conditions. We should only be attempting to resize TTY on running containers, but the containers can go from running to stopped at any time, and returning an error to the caller is just causing noice. This change will basically ignore requests to resize terminals if the container is not running and return the caller to success. All other callers will still return failure. Fixes: https://github.com/containers/podman/issues/9831 Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Diffstat (limited to 'pkg/api/server/register_exec.go')
-rw-r--r--pkg/api/server/register_exec.go5
1 files changed, 5 insertions, 0 deletions
diff --git a/pkg/api/server/register_exec.go b/pkg/api/server/register_exec.go
index 0f8c827c8..de437ab1a 100644
--- a/pkg/api/server/register_exec.go
+++ b/pkg/api/server/register_exec.go
@@ -136,6 +136,11 @@ func (s *APIServer) registerExecHandlers(r *mux.Router) error {
// name: w
// type: integer
// description: Width of the TTY session in characters
+ // - in: query
+ // name: running
+ // type: boolean
+ // required: false
+ // description: Ignore containers not running errors
// produces:
// - application/json
// responses: