-
Notifications
You must be signed in to change notification settings - Fork 362
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
manifest.GuessMIMEType(): recognize self-described OCI manifests #1419
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea, LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update also the comment below, something like mtrmac@665a421 .
(I’m not sure when I’ll finish that wider work, so merging this part now makes perfect sense.)
Oof, I need to add tests for this. |
3adb4be
to
1b8230e
Compare
Newer versions of the OCI spec include the mediaType field in image manifests and image indexes, and if they include them, save ourselves some work by using that information. Signed-off-by: Nalin Dahyabhai <nalin@redhat.com>
1b8230e
to
886a2ed
Compare
Added "nomime" variants to the existing OCI test cases for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM assuming tests pass. Thanks!
LGTM |
... to a version past 1.0.2, just to keep various dependency checkers happy. > go get github.com/opencontainers/image-spec@v1.0.2-0.20211123152302-43a7dee1ec31 The commit is intended to match containers/image#1419 to minimize churn. NOTE: This is NOT a fix for CVE-2021-41190 / GHSA-77vh-xpmg-72qh , that was fixed in Skopeo 1.5.2. Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Newer versions of the OCI spec include the mediaType field in image manifests and image indexes, and if they include them, save ourselves some work by using that information.