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

dannykopping · SasSwart · web-flow · commit e2a46393fce4 · 2025-11-27T17:43:39.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
@@ -1087,7 +1087,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
`@@ -1087,7 +1087,7 @@ func (a agent) handleManifest(manifestOK checkpoint) func(ctx context.Context,`
`1087`	`1087`	`if err != nil {`
`1088`	`1088`	`return xerrors.Errorf("fetch metadata: %w", err)`
`1089`	`1089`	`}`
`1090`		`- a.logger.Info(ctx, "fetched manifest", slog.F("manifest", mp))`
	`1090`	`+ a.logger.Info(ctx, "fetched manifest")`
`1091`	`1091`	`manifest, err := agentsdk.ManifestFromProto(mp)`
`1092`	`1092`	`if err != nil {`
`1093`	`1093`	`a.logger.Critical(ctx, "failed to convert manifest", slog.F("manifest", mp), slog.Error(err))`