Revert SSH conn handling

kylecarbs · kylecarbs · commit 9b0b80a5d698 · 2023-02-03T18:04:52.000Z
diff --git a/agent/agent.go b/agent/agent.go
@@ -645,24 +645,15 @@ func (a *agent) init(ctx context.Context) {
 			sshLogger.Info(ctx, "ssh connection ended", slog.Error(err))
 		},
 		Handler: func(session ssh.Session) {
-			err := a.trackConnGoroutine(func() {
-				err := a.handleSSHSession(session)
-				var exitError *exec.ExitError
-				if xerrors.As(err, &exitError) {
-					a.logger.Debug(ctx, "ssh session returned", slog.Error(exitError))
-					_ = session.Exit(exitError.ExitCode())
-					return
-				}
-				if err != nil {
-					a.logger.Warn(ctx, "ssh session failed", slog.Error(err))
-					// This exit code is designed to be unlikely to be confused for a legit exit code
-					// from the process.
-					_ = session.Exit(MagicSessionErrorCode)
-					return
-				}
-			})
+			err := a.handleSSHSession(session)
+			var exitError *exec.ExitError
+			if xerrors.As(err, &exitError) {
+				a.logger.Debug(ctx, "ssh session returned", slog.Error(exitError))
+				_ = session.Exit(exitError.ExitCode())
+				return
+			}
 			if err != nil {
-				a.logger.Warn(ctx, "track ssh session failed", slog.Error(err))
+				a.logger.Warn(ctx, "ssh session failed", slog.Error(err))
 				// This exit code is designed to be unlikely to be confused for a legit exit code
 				// from the process.
 				_ = session.Exit(MagicSessionErrorCode)
diff --git a/cli/root_test.go b/cli/root_test.go
@@ -121,7 +121,7 @@ ExtractCommandPathsLoop:
 
 			got := buf.Bytes()
 			// Remove CRLF newlines (Windows).
-			// got = bytes.ReplaceAll(got, []byte{'\r', '\n'}, []byte{'\n'})
+			got = bytes.ReplaceAll(got, []byte{'\r', '\n'}, []byte{'\n'})
 
 			// The `coder templates create --help` command prints the path
 			// to the working directory (--directory flag default value).
