Merge pull request #361 from ca-risken/fix-bug

fix: nil deriference
ca-risken · Jun 4, 2024 · 685857a · 685857a
2 parents 3900380 + 075dc73
commit 685857a
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/pkg/server/alert/alert_notification.go b/pkg/server/alert/alert_notification.go
@@ -234,6 +234,9 @@ func (a *AlertService) RequestProjectRoleNotification(ctx context.Context, req *
 	if err != nil {
 		return nil, err
 	}
+	if notifications == nil || len(*notifications) == 0 {
+		return nil, fmt.Errorf("notification not found. project_id: %d", req.ProjectId)
+	}
 	notification := (*notifications)[0]
 	projects, err := a.projectClient.ListProject(ctx, &project.ListProjectRequest{ProjectId: req.ProjectId})
 	if err != nil {