Skip to content

Commit

Permalink
httptransport: GET vuln report returns 404 when indexing in-progress
Browse files Browse the repository at this point in the history
Signed-off-by: Mark Frost <frostmar@uk.ibm.com>
  • Loading branch information
frostmar authored and crozzy committed Apr 29, 2024
1 parent e5a896c commit c6df986
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions httptransport/matcher_v1.go
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import (
"github.com/google/uuid"
"github.com/ldelossa/responserecorder"
"github.com/quay/claircore"
indexerController "github.com/quay/claircore/indexer/controller"
"github.com/quay/claircore/libvuln/driver"
"github.com/quay/zlog"
oteltrace "go.opentelemetry.io/contrib/instrumentation/net/http/httptrace/otelhttptrace"
Expand Down Expand Up @@ -119,8 +120,8 @@ func (h *MatcherV1) vulnerabilityReport(w http.ResponseWriter, r *http.Request)
apiError(ctx, w, http.StatusInternalServerError, "experienced a server side error: %v", err)
return
}
// now check bool only after confirming no err
if !ok {
// now check present and finished only after confirming no err
if !ok || indexReport.State != indexerController.IndexFinished.String() {
apiError(ctx, w, http.StatusNotFound, "index report for manifest %q not found", manifest.String())
return
}
Expand Down

0 comments on commit c6df986

Please sign in to comment.