fix: Add nil check for cronjobRef.Controller (#95)

ben-z · web-flow · commit 42f5d8f98c6e · 2024-07-30T14:52:22.000+02:00
Resolves #90
diff --git a/crons.go b/crons.go
@@ -35,7 +35,7 @@ func runSentryCronsCheckin(ctx context.Context, job *batchv1.Job, eventHandlerTy
 		return errors.New("job does not have cronjob reference")
 	}
 	cronjobRef := job.OwnerReferences[0]
-	if !*cronjobRef.Controller || cronjobRef.Kind != KindCronjob {
+	if cronjobRef.Controller == nil || !*cronjobRef.Controller || cronjobRef.Kind != KindCronjob {
 		return errors.New("job does not have cronjob reference")
 	}
 	cronsMonitorData, ok := cronsMetaData.getCronsMonitorData(cronjobRef.Name)

Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ func runSentryCronsCheckin(ctx context.Context, job *batchv1.Job, eventHandlerTy`
`35`	`35`	`return errors.New("job does not have cronjob reference")`
`36`	`36`	`}`
`37`	`37`	`cronjobRef := job.OwnerReferences[0]`
`38`		`- if !*cronjobRef.Controller \|\| cronjobRef.Kind != KindCronjob {`
	`38`	`+ if cronjobRef.Controller == nil \|\| !*cronjobRef.Controller \|\| cronjobRef.Kind != KindCronjob {`
`39`	`39`	`return errors.New("job does not have cronjob reference")`
`40`	`40`	`}`
`41`	`41`	`cronsMonitorData, ok := cronsMetaData.getCronsMonitorData(cronjobRef.Name)`