diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2021-03-22 10:17:24 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-22 10:17:24 -0700 |
| commit | d0bd775c02f0637b5775735c84438a01e8cf8395 (patch) | |
| tree | 6377971672dd4e4881c4a73f9626e0ccbcd1ddd0 /pkg/api/handlers/compat/containers_prune.go | |
| parent | 2cd37edf02cca715df41ec915bbe0d6cfdc720e9 (diff) | |
| parent | 907b34c46fe474911d4321795f029583e8daeeec (diff) | |
| download | podman-d0bd775c02f0637b5775735c84438a01e8cf8395.tar.gz podman-d0bd775c02f0637b5775735c84438a01e8cf8395.tar.bz2 podman-d0bd775c02f0637b5775735c84438a01e8cf8395.zip | |
Merge pull request #9773 from jmguzik/containers-prune-filters-http-api
Fix containers list/prune http api filter behaviour
Diffstat (limited to 'pkg/api/handlers/compat/containers_prune.go')
| -rw-r--r-- | pkg/api/handlers/compat/containers_prune.go | 17 |
1 files changed, 7 insertions, 10 deletions
diff --git a/pkg/api/handlers/compat/containers_prune.go b/pkg/api/handlers/compat/containers_prune.go index dc4d53af6..e37929d27 100644 --- a/pkg/api/handlers/compat/containers_prune.go +++ b/pkg/api/handlers/compat/containers_prune.go @@ -9,23 +9,20 @@ import ( "github.com/containers/podman/v3/pkg/api/handlers/utils" "github.com/containers/podman/v3/pkg/domain/entities/reports" "github.com/containers/podman/v3/pkg/domain/filters" - "github.com/gorilla/schema" + "github.com/containers/podman/v3/pkg/util" "github.com/pkg/errors" ) func PruneContainers(w http.ResponseWriter, r *http.Request) { runtime := r.Context().Value("runtime").(*libpod.Runtime) - decoder := r.Context().Value("decoder").(*schema.Decoder) - - query := struct { - Filters map[string][]string `schema:"filters"` - }{} - if err := decoder.Decode(&query, r.URL.Query()); err != nil { - utils.Error(w, "Something went wrong.", http.StatusBadRequest, errors.Wrapf(err, "failed to parse parameters for %s", r.URL.String())) + filtersMap, err := util.PrepareFilters(r) + if err != nil { + utils.Error(w, "Something went wrong.", http.StatusInternalServerError, errors.Wrapf(err, "failed to parse parameters for %s", r.URL.String())) return } - filterFuncs := make([]libpod.ContainerFilter, 0, len(query.Filters)) - for k, v := range query.Filters { + + filterFuncs := make([]libpod.ContainerFilter, 0, len(*filtersMap)) + for k, v := range *filtersMap { generatedFunc, err := filters.GenerateContainerFilterFuncs(k, v, runtime) if err != nil { utils.InternalServerError(w, err) |