-
Notifications
You must be signed in to change notification settings - Fork 210
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Question: What is the correct value for the "OCI-Filters-Applied" header? #448
Comments
That's a bug in the conformance test, it should be the hard coded string. |
"If multiple filters are applied, the header MUST contain a comma separated list of applied filters." ^ this language seems to indicate that (in future probably) one could ask for "artifactType=A&artifactType=B..." etc and the response header returns what then? If we were looking for a boolean, then just the presence of the header key should suffice? |
We haven't added additional filter types yet, but the note there is for clients to expect a comma separated list. In the future, there could be options to filter on annotation, sort, and limit the number of responses (e.g. return the most recent signature artifact signed by a given fingerprint). Multiple artifactType parameters are not currently supported. |
I see, so the idea was to support multiple filter types? artifactType being one of them, maybe "annotation=X" is another. Revisiting my earlier question for this specific endpoint, |
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]>
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]>
At present, that's undefined. I could see some registries applying both filters with either |
* 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]> * ci(conformance): point to temporary repo Revert this commit once upstream changes have been merged Signed-off-by: Ramkumar Chinchani <[email protected]> --------- Signed-off-by: Ramkumar Chinchani <[email protected]>
Got it! Thanks @sudo-bmitch @rchincha for taking care of this! |
I have a question about the "Listing Referrers" section of the distribution-spec
v1.1.0-rc3
, which states that:Question
From the above statement, it appears that the value of the
OCI-Filters-Applied
header should be a hard-coded stringartifactType
, rather than the concrete artifact type (such asapplication/vnd.example.sbom.v1
in this example).However, the conformance test checks the concrete artifact type (
application/vnd.example.sbom.v1
in this example) instead of the stringartifactType
. This seems to contradict the spec.distribution-spec/conformance/03_discovery_test.go
Lines 323 to 326 in 42d2d6e
Could someone please help clarify what the correct value of the
OCI-Filters-Applied
header should be? Should it be the hard-coded stringartifactType
or the concrete artifact type?Thank you!
The text was updated successfully, but these errors were encountered: