Log unmarshal failure

code-asher · code-asher · commit 4d654782c420 · 2023-12-01T12:14:46.000-09:00
diff --git a/agent/agentssh/jetbrainstrack.go b/agent/agentssh/jetbrainstrack.go
@@ -31,21 +31,19 @@ func NewJetbrainsChannelWatcher(ctx ssh.Context, logger slog.Logger, newChannel
 	d := localForwardChannelData{}
 	if err := gossh.Unmarshal(newChannel.ExtraData(), &d); err != nil {
 		// If the data fails to unmarshal, do nothing.
+		logger.Warn(ctx, "failed to unmarshal port forward data", slog.Error(err))
 		return newChannel
 	}
 
 	// If we do get a port, we should be able to get the matching PID and from
 	// there look up the invocation.
 	cmdline, err := getListeningPortProcessCmdline(d.DestPort)
 	if err != nil {
-		logger.Warn(ctx, "port inspection failed",
+		logger.Warn(ctx, "failed to inspect port",
 			slog.F("destination_port", d.DestPort),
 			slog.Error(err))
 		return newChannel
 	}
-	logger.Debug(ctx, "checking forwarded process",
-		slog.F("cmdline", cmdline),
-		slog.F("destination_port", d.DestPort))
 
 	// If this is not JetBrains, then we do not need to do anything special.  We
 	// attempt to match on something that appears unique to JetBrains software and
