Code review comments

priyawadhwa · priyawadhwa · commit a8bff9985acd · 2022-09-08T07:40:13.000-07:00
Signed-off-by: Priya Wadhwa &lt;priya@chainguard.dev&gt;
diff --git a/pkg/types/intoto/intoto.go b/pkg/types/intoto/intoto.go
@@ -82,12 +82,12 @@ func (it *BaseIntotoType) CreateProposedEntry(ctx context.Context, version strin
 			}
 			ei, err := it.VersionedUnmarshal(nil, v)
 			if err != nil {
-				log.Logger.Errorf("fetching Intoto version (%v) implementation: %w", v, err)
+				log.ContextLogger(ctx).Errorf("fetching Intoto version (%v) implementation: %w", v, err)
 				continue
 			}
 			versionedPE, err := ei.CreateFromArtifactProperties(ctx, props)
 			if err != nil {
-				log.Logger.Errorf("error creating Intoto entry of version (%v): %w", v, err)
+				log.ContextLogger(ctx).Errorf("error creating Intoto entry of version (%v): %w", v, err)
 				continue
 			}
 			next.next = &ProposedIntotoEntryIterator{versionedPE, nil}

Original file line number	Diff line number	Diff line change
`@@ -82,12 +82,12 @@ func (it *BaseIntotoType) CreateProposedEntry(ctx context.Context, version strin`
`82`	`82`	`}`
`83`	`83`	`ei, err := it.VersionedUnmarshal(nil, v)`
`84`	`84`	`if err != nil {`
`85`		`- log.Logger.Errorf("fetching Intoto version (%v) implementation: %w", v, err)`
	`85`	`+ log.ContextLogger(ctx).Errorf("fetching Intoto version (%v) implementation: %w", v, err)`
`86`	`86`	`continue`
`87`	`87`	`}`
`88`	`88`	`versionedPE, err := ei.CreateFromArtifactProperties(ctx, props)`
`89`	`89`	`if err != nil {`
`90`		`- log.Logger.Errorf("error creating Intoto entry of version (%v): %w", v, err)`
	`90`	`+ log.ContextLogger(ctx).Errorf("error creating Intoto entry of version (%v): %w", v, err)`
`91`	`91`	`continue`
`92`	`92`	`}`
`93`	`93`	`next.next = &ProposedIntotoEntryIterator{versionedPE, nil}`