From 8cc743a812baa29ad52af2aea2440a8e7b97429f Mon Sep 17 00:00:00 2001 From: Aaron Lehmann Date: Wed, 16 Apr 2025 02:44:33 -0700 Subject: [PATCH] chore: clarify error variable name in doAttach (#17284) --- agent/reconnectingpty/screen.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/agent/reconnectingpty/screen.go b/agent/reconnectingpty/screen.go index 533c11a06b..04e1861ead 100644 --- a/agent/reconnectingpty/screen.go +++ b/agent/reconnectingpty/screen.go @@ -307,9 +307,9 @@ func (rpty *screenReconnectingPTY) doAttach(ctx context.Context, conn net.Conn, if closeErr != nil { logger.Debug(ctx, "closed ptty with error", slog.Error(closeErr)) } - closeErr = process.Kill() - if closeErr != nil { - logger.Debug(ctx, "killed process with error", slog.Error(closeErr)) + killErr := process.Kill() + if killErr != nil { + logger.Debug(ctx, "killed process with error", slog.Error(killErr)) } rpty.metrics.WithLabelValues("screen_wait").Add(1) return nil, nil, err