diff options
| author | baude <bbaude@redhat.com> | 2019-05-21 09:10:23 -0500 |
|---|---|---|
| committer | baude <bbaude@redhat.com> | 2019-05-21 09:13:19 -0500 |
| commit | 00262edf6c89840f8b90db7d8e7e664138478eda (patch) | |
| tree | 8af3362541a37878c04ff093c4ae1c5ac7bb86c0 | |
| parent | 773b9ac9d059a66280ac50163a17bc705bde0a60 (diff) | |
| download | podman-00262edf6c89840f8b90db7d8e7e664138478eda.tar.gz podman-00262edf6c89840f8b90db7d8e7e664138478eda.tar.bz2 podman-00262edf6c89840f8b90db7d8e7e664138478eda.zip | |
make remote resize channel buffered
when doing any sort of attach to a container, a sigwinch is sent
followed by a resize event. this is fine for the local client but when
doing things over the varlink, the first sigwinch is wiped out by the
immediate resize event and is therefore lost. by making the channel
buffered, both events are processed after the varlink connection is
established.
Signed-off-by: baude <bbaude@redhat.com>
| -rw-r--r-- | pkg/adapter/containers_remote.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/adapter/containers_remote.go b/pkg/adapter/containers_remote.go index 63b0f9d2f..c34495b3d 100644 --- a/pkg/adapter/containers_remote.go +++ b/pkg/adapter/containers_remote.go @@ -561,7 +561,7 @@ func (r *LocalRuntime) attach(ctx context.Context, stdin, stdout *os.File, cid s if err != nil { return nil, err } - resize := make(chan remotecommand.TerminalSize) + resize := make(chan remotecommand.TerminalSize, 5) haveTerminal := terminal.IsTerminal(int(os.Stdin.Fd())) // Check if we are attached to a terminal. If we are, generate resize |