From 24408263d994e911834fc3eb06f054a9c19332ac Mon Sep 17 00:00:00 2001 From: Aaron Lehmann Date: Fri, 24 Jul 2015 10:42:02 -0700 Subject: [PATCH] Manifest PUT should return 201 Created Change handler, update descriptors table, regenerate API spec, and update test. Signed-off-by: Aaron Lehmann --- docs/api/v2/descriptors.go | 2 +- docs/handlers/api_test.go | 4 ++-- docs/handlers/images.go | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/docs/api/v2/descriptors.go b/docs/api/v2/descriptors.go index f2551ffe..d6e37d05 100644 --- a/docs/api/v2/descriptors.go +++ b/docs/api/v2/descriptors.go @@ -523,7 +523,7 @@ var routeDescriptors = []RouteDescriptor{ Successes: []ResponseDescriptor{ { Description: "The manifest has been accepted by the registry and is stored under the specified `name` and `tag`.", - StatusCode: http.StatusAccepted, + StatusCode: http.StatusCreated, Headers: []ParameterDescriptor{ { Name: "Location", diff --git a/docs/handlers/api_test.go b/docs/handlers/api_test.go index 8d631941..62a03b62 100644 --- a/docs/handlers/api_test.go +++ b/docs/handlers/api_test.go @@ -423,7 +423,7 @@ func TestManifestAPI(t *testing.T) { checkErr(t, err, "building manifest url") resp = putManifest(t, "putting signed manifest", manifestURL, signedManifest) - checkResponse(t, "putting signed manifest", resp, http.StatusAccepted) + checkResponse(t, "putting signed manifest", resp, http.StatusCreated) checkHeaders(t, resp, http.Header{ "Location": []string{manifestDigestURL}, "Docker-Content-Digest": []string{dgst.String()}, @@ -432,7 +432,7 @@ func TestManifestAPI(t *testing.T) { // -------------------- // Push by digest -- should get same result resp = putManifest(t, "putting signed manifest", manifestDigestURL, signedManifest) - checkResponse(t, "putting signed manifest", resp, http.StatusAccepted) + checkResponse(t, "putting signed manifest", resp, http.StatusCreated) checkHeaders(t, resp, http.Header{ "Location": []string{manifestDigestURL}, "Docker-Content-Digest": []string{dgst.String()}, diff --git a/docs/handlers/images.go b/docs/handlers/images.go index e5b0bc77..61eac69e 100644 --- a/docs/handlers/images.go +++ b/docs/handlers/images.go @@ -183,7 +183,7 @@ func (imh *imageManifestHandler) PutImageManifest(w http.ResponseWriter, r *http w.Header().Set("Location", location) w.Header().Set("Docker-Content-Digest", imh.Digest.String()) - w.WriteHeader(http.StatusAccepted) + w.WriteHeader(http.StatusCreated) } // DeleteImageManifest removes the image with the given tag from the registry.