Skip to content

Commit

Permalink
Merge pull request #72 from layer5io/kumarabd/feature/exec
Browse files Browse the repository at this point in the history
fixes for stale terminal sessions
  • Loading branch information
kumarabd authored Jun 28, 2021
2 parents da057f0 + 8d25818 commit 49e040e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
1 change: 0 additions & 1 deletion go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ module github.com/layer5io/meshsync
replace (
github.com/docker/docker => github.com/moby/moby v17.12.0-ce-rc1.0.20200618181300-9dc6525e6118+incompatible
github.com/kudobuilder/kuttl => github.com/layer5io/kuttl v0.4.1-0.20200806180306-b7e46afd657f
github.com/layer5io/meshkit v0.2.12 => ../meshkit
golang.org/x/sys => golang.org/x/sys v0.0.0-20200826173525-f9321e4c35a6
vbom.ml/util => github.com/fvbommel/util v0.0.0-20180919145318-efcd4e0f9787
)
Expand Down
4 changes: 4 additions & 0 deletions meshsync/exec.go
Original file line number Diff line number Diff line change
Expand Up @@ -133,6 +133,10 @@ func (h *Handler) streamSession(id string, req model.ExecRequest, cfg config.Lis
}()

for {
if _, ok := h.channelPool[id]; !ok {
h.Log.Info("Session closed for: ", id)
return
}
select {
case msg := <-subCh:
if msg.ObjectType == broker.ExecInputObject {
Expand Down

0 comments on commit 49e040e

Please sign in to comment.