Merge pull request #233 from xuzhu-591/main

fix: nil point panic
horizoncd · Jul 4, 2024 · e60ee39 · e60ee39
2 parents fbca5de + 8d365d3
commit e60ee39
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/eventhandler/wlgenerator/wlgenerator.go b/pkg/eventhandler/wlgenerator/wlgenerator.go
@@ -188,7 +188,7 @@ func (w *WebhookLogGenerator) listAssociatedResourcesOfCluster(ctx context.Conte
 func (w *WebhookLogGenerator) listAssociatedResourcesOfPipelinerun(ctx context.Context,
 	id uint) (*prmodels.Pipelinerun, *clustermodels.Cluster, map[string][]uint) {
 	pr, err := w.prMgr.PipelineRun.GetByID(ctx, id)
-	if err != nil {
+	if err != nil || pr == nil {
 		log.Warningf(ctx, "pipelinerun %d is not exist",
 			id)
 		return nil, nil, nil