diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2021-02-10 01:45:31 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-02-10 01:45:31 -0500 |
| commit | 055e2dda3a1888d319d542abe8735e791e736451 (patch) | |
| tree | 31fa336a7dd1301780fe2ab2e863c373e23f8033 | |
| parent | 2d829aeb1fe0dc19c45e796141ab78ff192b5dbb (diff) | |
| parent | f2a856203117bf97a8dfa5960468ae70e4526e3e (diff) | |
| download | podman-055e2dda3a1888d319d542abe8735e791e736451.tar.gz podman-055e2dda3a1888d319d542abe8735e791e736451.tar.bz2 podman-055e2dda3a1888d319d542abe8735e791e736451.zip | |
Merge pull request #9295 from Luap99/fix-9293
Fix compat networks endpoint for a empty result
| -rw-r--r-- | pkg/api/handlers/compat/networks.go | 2 | ||||
| -rw-r--r-- | test/apiv2/35-networks.at | 3 |
2 files changed, 4 insertions, 1 deletions
diff --git a/pkg/api/handlers/compat/networks.go b/pkg/api/handlers/compat/networks.go index f7a70816f..85d2db87e 100644 --- a/pkg/api/handlers/compat/networks.go +++ b/pkg/api/handlers/compat/networks.go @@ -196,7 +196,7 @@ func ListNetworks(w http.ResponseWriter, r *http.Request) { return } - var reports []*types.NetworkResource + reports := []*types.NetworkResource{} logrus.Debugf("netNames: %q", strings.Join(netNames, ", ")) for _, name := range netNames { report, err := getNetworkResourceByNameOrID(name, runtime, query.Filters) diff --git a/test/apiv2/35-networks.at b/test/apiv2/35-networks.at index 5327bd076..7ce109913 100644 --- a/test/apiv2/35-networks.at +++ b/test/apiv2/35-networks.at @@ -46,6 +46,9 @@ length=1 \ # invalid filter filters={"dangling":["1"]} t GET networks?filters=%7B%22dangling%22%3A%5B%221%22%5D%7D 500 \ .cause='invalid filter "dangling"' +# (#9293 with no networks the endpoint should return empty array instead of null) +t GET networks?filters=%7B%22name%22%3A%5B%22doesnotexists%22%5D%7D 200 \ +"[]" # network inspect docker t GET networks/a7662f44d65029fd4635c91feea3d720a57cef52e2a9fcc7772b69072cc1ccd1 200 \ |