diff options
author | Valentin Rothberg <rothberg@redhat.com> | 2021-12-02 13:25:54 +0100 |
---|---|---|
committer | Valentin Rothberg <rothberg@redhat.com> | 2021-12-02 14:29:17 +0100 |
commit | 2683ecbbcda750b9339713ced9b974b9dd61b9aa (patch) | |
tree | 688253d18590ae0fdf4fa110207e2bfc18d4e0b7 /pkg/api/handlers/compat/images.go | |
parent | eeb71490e57047202410821fe98fa4332371cb03 (diff) | |
download | podman-2683ecbbcda750b9339713ced9b974b9dd61b9aa.tar.gz podman-2683ecbbcda750b9339713ced9b974b9dd61b9aa.tar.bz2 podman-2683ecbbcda750b9339713ced9b974b9dd61b9aa.zip |
compat: images/json
Do not list manifest lists. Docker doesn't either.
Fixes: #12453
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
Diffstat (limited to 'pkg/api/handlers/compat/images.go')
-rw-r--r-- | pkg/api/handlers/compat/images.go | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/pkg/api/handlers/compat/images.go b/pkg/api/handlers/compat/images.go index af8b6b63d..4533fddeb 100644 --- a/pkg/api/handlers/compat/images.go +++ b/pkg/api/handlers/compat/images.go @@ -409,14 +409,20 @@ func GetImages(w http.ResponseWriter, r *http.Request) { utils.Error(w, "Something went wrong.", http.StatusInternalServerError, errors.Wrap(err, "Failed get images")) return } - var summaries = make([]*entities.ImageSummary, len(images)) - for j, img := range images { + + summaries := make([]*entities.ImageSummary, 0, len(images)) + for _, img := range images { + // If the image is a manifest list, extract as much as we can. + if isML, _ := img.IsManifestList(r.Context()); isML { + continue + } + is, err := handlers.ImageToImageSummary(img) if err != nil { utils.Error(w, "Something went wrong.", http.StatusInternalServerError, errors.Wrap(err, "Failed transform image summaries")) return } - summaries[j] = is + summaries = append(summaries, is) } utils.WriteResponse(w, http.StatusOK, summaries) } |