diff --git a/internal/servers/worker/session.go b/internal/servers/worker/session.go index e387df6a80..87fd147552 100644 --- a/internal/servers/worker/session.go +++ b/internal/servers/worker/session.go @@ -296,7 +296,8 @@ func (w *Worker) closeConnections(ctx context.Context, closeInfo map[string]stri if err != nil { w.logger.Error(err.Error()) - w.logger.Error("serious error in processing return data from controller, aborting marking connections as closed") + w.logger.Error("serious error in processing return data from controller, aborting additional session/connection state modification") + return } // Mark connections as closed diff --git a/internal/servers/worker/session_test.go b/internal/servers/worker/session_test.go index 58f2559d5f..750eecfdb4 100644 --- a/internal/servers/worker/session_test.go +++ b/internal/servers/worker/session_test.go @@ -79,7 +79,7 @@ func TestMakeFakeSessionCloseInfo(t *testing.T) { require.Equal(expected, actual) } -func TestMakeFakeSessionCloseInfoPanicIfCloseInfoNil(t *testing.T) { +func TestMakeFakeSessionCloseInfoErrorIfCloseInfoNil(t *testing.T) { require := require.New(t) actual, err := new(Worker).makeFakeSessionCloseInfo(nil) require.Nil(actual)