diff options
| author | Paul Holzinger <pholzing@redhat.com> | 2021-06-03 16:07:43 +0200 |
|---|---|---|
| committer | Paul Holzinger <pholzing@redhat.com> | 2021-06-04 16:55:48 +0200 |
| commit | 1f73374acd3c33d1884e9383205c9f891a3552db (patch) | |
| tree | 95ed36dbd4022c772a3be26d2c8205ba9ec6507c /pkg/api/server | |
| parent | 52dae693da0df1447b7f5210a4c842d5c5a8a401 (diff) | |
| download | podman-1f73374acd3c33d1884e9383205c9f891a3552db.tar.gz podman-1f73374acd3c33d1884e9383205c9f891a3552db.tar.bz2 podman-1f73374acd3c33d1884e9383205c9f891a3552db.zip | |
remote: always send resize before the container starts
There is race condition in the remote client attach logic. Because the
resize api call was handled in an extra goroutine the container was
started before the resize call happend. To fix this we have to call
resize in the same goroutine as attach. When the first resize is done
start a goroutine to listen on SIGWINCH in the background and resize
again if the signal is received.
Fixes #9859
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Diffstat (limited to 'pkg/api/server')
| -rw-r--r-- | pkg/api/server/register_containers.go | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/pkg/api/server/register_containers.go b/pkg/api/server/register_containers.go index aa999905e..88ebb4df5 100644 --- a/pkg/api/server/register_containers.go +++ b/pkg/api/server/register_containers.go @@ -1364,6 +1364,8 @@ func (s *APIServer) registerContainersHandlers(r *mux.Router) error { // $ref: "#/responses/ok" // 404: // $ref: "#/responses/NoSuchContainer" + // 409: + // $ref: "#/responses/ConflictError" // 500: // $ref: "#/responses/InternalError" r.HandleFunc(VersionedPath("/libpod/containers/{name}/resize"), s.APIHandler(compat.ResizeTTY)).Methods(http.MethodPost) |