diff options
| author | Ed Santiago <santiago@redhat.com> | 2020-02-11 09:13:53 -0700 |
|---|---|---|
| committer | Ed Santiago <santiago@redhat.com> | 2020-02-11 10:50:23 -0700 |
| commit | bdccdd2265340d48a0ba868d8c312fdd3172f1c3 (patch) | |
| tree | 631919c602a8df61b1f53ba6844095d683d6268c /pkg/api/server | |
| parent | 86b5a89d1afb641196214ed9c57e83e617776c5f (diff) | |
| download | podman-bdccdd2265340d48a0ba868d8c312fdd3172f1c3.tar.gz podman-bdccdd2265340d48a0ba868d8c312fdd3172f1c3.tar.bz2 podman-bdccdd2265340d48a0ba868d8c312fdd3172f1c3.zip | |
API v2: pods: fix two incorrect return codes
1) /pods/<X>/exists - is documented to return 204, and that's
the correct value, but until now it has been returning 200.
2) /pods/create - return 409 (conflict), not 500, when pod
already exists
Also: in WriteResponse(), if code is 204 (No Content) or 304
(Not Modified), emit the status code only but no content-type
headers nor content.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'pkg/api/server')
| -rw-r--r-- | pkg/api/server/register_pods.go | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/pkg/api/server/register_pods.go b/pkg/api/server/register_pods.go index 5c7b51871..974568d47 100644 --- a/pkg/api/server/register_pods.go +++ b/pkg/api/server/register_pods.go @@ -41,6 +41,8 @@ func (s *APIServer) registerPodsHandlers(r *mux.Router) error { // type: string // 400: // $ref: "#/responses/BadParamError" + // 409: + // description: pod already exists // 500: // $ref: "#/responses/InternalError" r.Handle(VersionedPath("/libpod/pods/prune"), APIHandler(s.Context, libpod.PodPrune)).Methods(http.MethodPost) |