Skip to content

Commit

Permalink
fix: use ComplianceID as summary id
Browse files Browse the repository at this point in the history
  • Loading branch information
artaasadi committed Dec 26, 2024
1 parent dfa7f19 commit 305af1e
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions services/compliance/http_routes.go
Original file line number Diff line number Diff line change
Expand Up @@ -7880,8 +7880,7 @@ func (h HttpHandler) GetComplianceJobReport(c echo.Context) error {
jobId = strconv.Itoa(int(*complianceJob.SummaryJobId))
}

summary, err := es.GetJobReportControlSummaryByJobID(c.Request().Context(), h.logger, h.client,
jobId, complianceJob.WithIncidents, controls)
summary, err := es.GetJobReportControlSummaryByJobID(c.Request().Context(), h.logger, h.client, jobId, controls)
if err != nil {
h.logger.Error("failed to get job report control summary by job id", zap.Error(err))
return echo.NewHTTPError(http.StatusInternalServerError, "failed to get job report control summary by job id")
Expand Down Expand Up @@ -7939,8 +7938,7 @@ func (h HttpHandler) GetJobReportSummary(ctx echo.Context) error {
}
controls, err := h.db.ListControls(ctx.Request().Context(), controlsStr, nil)

summary, err := es.GetJobReportControlSummaryByJobID(ctx.Request().Context(), h.logger, h.client,
jobId, controlsFilter)
summary, err := es.GetJobReportControlSummaryByJobID(ctx.Request().Context(), h.logger, h.client, jobId, controlsFilter)
if err != nil {
h.logger.Error("failed to get job report control summary by job id", zap.Error(err))
return echo.NewHTTPError(http.StatusInternalServerError, "failed to get job report control summary by job id")
Expand Down

0 comments on commit 305af1e

Please sign in to comment.