fix: correct time import and FinishedOn bugs in SLSA parser

Abhishek9639 · Abhishek9639 · commit 975101ec2abe · 2026-05-05T18:35:34.000+05:30
Signed-off-by: Abhishek &lt;abhishekup082@gmail.com&gt;
diff --git a/pkg/ingestor/parser/slsa/parser_slsa.go b/pkg/ingestor/parser/slsa/parser_slsa.go
@@ -20,6 +20,7 @@ import (
 	"errors"
 	"fmt"
 	"strings"
+	"time"
 
 	jsoniter "github.com/json-iterator/go"
 
@@ -258,7 +259,7 @@ func fillSLSA02(inp *model.SLSAInputSpec, pred *ProvenancePredicateV02) error {
 		inp.StartedOn = pred.Metadata.BuildStartedOn
 	}
 	if pred.Metadata.BuildFinishedOn != nil {
-		inp.FinishedOn = pred.Metadata.BuildStartedOn
+		inp.FinishedOn = pred.Metadata.BuildFinishedOn
 	}
 	return nil
 }
@@ -273,7 +274,7 @@ func fillSLSA1(inp *model.SLSAInputSpec, pred *slsa1.Provenance) error {
 		inp.StartedOn = &startTimePB
 	}
 	if pred.RunDetails.Metadata.FinishedOn != nil {
-		finishTimePB := time.Unix(pred.RunDetails.Metadata.StartedOn.GetSeconds(), int64(pred.RunDetails.Metadata.StartedOn.GetNanos()))
+		finishTimePB := time.Unix(pred.RunDetails.Metadata.FinishedOn.GetSeconds(), int64(pred.RunDetails.Metadata.FinishedOn.GetNanos()))
 		inp.FinishedOn = &finishTimePB
 	}
 	return nil

Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@ import (`
`20`	`20`	`"errors"`
`21`	`21`	`"fmt"`
`22`	`22`	`"strings"`
	`23`	`+ "time"`
`23`	`24`
`24`	`25`	`jsoniter "github.com/json-iterator/go"`
`25`	`26`
`@@ -258,7 +259,7 @@ func fillSLSA02(inp model.SLSAInputSpec, pred ProvenancePredicateV02) error {`
`258`	`259`	`inp.StartedOn = pred.Metadata.BuildStartedOn`
`259`	`260`	`}`
`260`	`261`	`if pred.Metadata.BuildFinishedOn != nil {`
`261`		`- inp.FinishedOn = pred.Metadata.BuildStartedOn`
	`262`	`+ inp.FinishedOn = pred.Metadata.BuildFinishedOn`
`262`	`263`	`}`
`263`	`264`	`return nil`
`264`	`265`	`}`
`@@ -273,7 +274,7 @@ func fillSLSA1(inp model.SLSAInputSpec, pred slsa1.Provenance) error {`
`273`	`274`	`inp.StartedOn = &startTimePB`
`274`	`275`	`}`
`275`	`276`	`if pred.RunDetails.Metadata.FinishedOn != nil {`
`276`		`- finishTimePB := time.Unix(pred.RunDetails.Metadata.StartedOn.GetSeconds(), int64(pred.RunDetails.Metadata.StartedOn.GetNanos()))`
	`277`	`+ finishTimePB := time.Unix(pred.RunDetails.Metadata.FinishedOn.GetSeconds(), int64(pred.RunDetails.Metadata.FinishedOn.GetNanos()))`
`277`	`278`	`inp.FinishedOn = &finishTimePB`
`278`	`279`	`}`
`279`	`280`	`return nil`