diff options
| author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2021-03-30 21:29:26 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-30 21:29:26 +0200 |
| commit | bd07179e9c608ff1075f052843aeb9b1c8b7c7e3 (patch) | |
| tree | c99b7a41571e0b7593afe0a197747685e4670b66 | |
| parent | 989c8e97851c1a443270e803e19dd069fc199302 (diff) | |
| parent | eef66973de4738536810be71b91e3503e7aa5183 (diff) | |
| download | podman-bd07179e9c608ff1075f052843aeb9b1c8b7c7e3.tar.gz podman-bd07179e9c608ff1075f052843aeb9b1c8b7c7e3.tar.bz2 podman-bd07179e9c608ff1075f052843aeb9b1c8b7c7e3.zip | |
Merge pull request #9892 from jwhonce/wip/top
Trim white space from /top endpoint results
| -rw-r--r-- | pkg/api/handlers/compat/containers_top.go | 10 | ||||
| -rw-r--r-- | test/apiv2/25-containersMore.at | 3 |
2 files changed, 11 insertions, 2 deletions
diff --git a/pkg/api/handlers/compat/containers_top.go b/pkg/api/handlers/compat/containers_top.go index ab9f613af..cae044a89 100644 --- a/pkg/api/handlers/compat/containers_top.go +++ b/pkg/api/handlers/compat/containers_top.go @@ -46,8 +46,16 @@ func TopContainer(w http.ResponseWriter, r *http.Request) { var body = handlers.ContainerTopOKBody{} if len(output) > 0 { body.Titles = strings.Split(output[0], "\t") + for i := range body.Titles { + body.Titles[i] = strings.TrimSpace(body.Titles[i]) + } + for _, line := range output[1:] { - body.Processes = append(body.Processes, strings.Split(line, "\t")) + process := strings.Split(line, "\t") + for i := range process { + process[i] = strings.TrimSpace(process[i]) + } + body.Processes = append(body.Processes, process) } } utils.WriteJSON(w, http.StatusOK, body) diff --git a/test/apiv2/25-containersMore.at b/test/apiv2/25-containersMore.at index 39bfa2e32..0a049d869 100644 --- a/test/apiv2/25-containersMore.at +++ b/test/apiv2/25-containersMore.at @@ -38,7 +38,8 @@ t GET libpod/containers/foo/json 200 \ # List processes of the container t GET libpod/containers/foo/top 200 \ - length=2 + length=2 \ + .Processes[0][7]="top" # List processes of none such t GET libpod/containers/nonesuch/top 404 |