fix: remove a sensitive field from an agent log line (#20968) (#20970)

dannykopping · SasSwart · web-flow · commit a75205a55921 · 2025-11-27T17:35:00.000+02:00
Backport of #20968 Co-authored-by: Sas Swart <sas.swart.cdk@gmail.com>
diff --git a/agent/agent.go b/agent/agent.go
@@ -1097,7 +1097,7 @@ func (a *agent) handleManifest(manifestOK *checkpoint) func(ctx context.Context,
 		if err != nil {
 			return xerrors.Errorf("fetch metadata: %w", err)
 		}
-		a.logger.Info(ctx, "fetched manifest", slog.F("manifest", mp))
+		a.logger.Info(ctx, "fetched manifest")
 		manifest, err := agentsdk.ManifestFromProto(mp)
 		if err != nil {
 			a.logger.Critical(ctx, "failed to convert manifest", slog.F("manifest", mp), slog.Error(err))

Original file line number	Diff line number	Diff line change
`@@ -1097,7 +1097,7 @@ func (a agent) handleManifest(manifestOK checkpoint) func(ctx context.Context,`
`1097`	`1097`	`if err != nil {`
`1098`	`1098`	`return xerrors.Errorf("fetch metadata: %w", err)`
`1099`	`1099`	`}`
`1100`		`- a.logger.Info(ctx, "fetched manifest", slog.F("manifest", mp))`
	`1100`	`+ a.logger.Info(ctx, "fetched manifest")`
`1101`	`1101`	`manifest, err := agentsdk.ManifestFromProto(mp)`
`1102`	`1102`	`if err != nil {`
`1103`	`1103`	`a.logger.Critical(ctx, "failed to convert manifest", slog.F("manifest", mp), slog.Error(err))`