fix(conformance): OCI-Filters-Applied should return a literal

opencontainers/distribution-spec#448 Should only indicate what filter-type was applied and not what exact values it was filtered on. Signed-off-by: Ramkumar Chinchani <[email protected]>
rchincha · Jul 21, 2023 · 6c79914 · 6c79914
1 parent 46ee330
commit 6c79914
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pkg/api/controller_test.go b/pkg/api/controller_test.go
@@ -5924,15 +5924,15 @@ func TestArtifactReferences(t *testing.T) {
 				So(err, ShouldBeNil)
 				So(resp.StatusCode(), ShouldEqual, http.StatusOK)
 				So(resp.Header().Get("Content-Type"), ShouldEqual, ispec.MediaTypeImageIndex)
-				So(resp.Header().Get("OCI-Filters-Applied"), ShouldEqual, artifactType)
+				So(resp.Header().Get("OCI-Filters-Applied"), ShouldEqual, "artifactType")
 
 				resp, err = resty.R().SetQueryParams(map[string]string{"artifactType": artifactType +
 					",otherArtType"}).Get(baseURL + fmt.Sprintf("/v2/%s/referrers/%s", repoName,
 					digest.String()))
 				So(err, ShouldBeNil)
 				So(resp.StatusCode(), ShouldEqual, http.StatusOK)
 				So(resp.Header().Get("Content-Type"), ShouldEqual, ispec.MediaTypeImageIndex)
-				So(resp.Header().Get("OCI-Filters-Applied"), ShouldEqual, artifactType+",otherArtType")
+				So(resp.Header().Get("OCI-Filters-Applied"), ShouldEqual, "artifactType")
 			})
 		})
 	})

diff --git a/pkg/api/routes.go b/pkg/api/routes.go
@@ -624,7 +624,8 @@ func (rh *RouteHandler) GetReferrers(response http.ResponseWriter, request *http
 	}
 
 	if len(artifactTypes) > 0 {
-		response.Header().Set("OCI-Filters-Applied", strings.Join(artifactTypes, ","))
+		// currently, the only filter supported and on this end-point
+		response.Header().Set("OCI-Filters-Applied", "artifactType")
 	}
 
 	zcommon.WriteData(response, http.StatusOK, ispec.MediaTypeImageIndex, out)