summaryrefslogtreecommitdiff
path: root/pkg/api/handlers
diff options
context:
space:
mode:
authorMatej Vasek <mvasek@redhat.com>2021-11-28 19:02:15 +0100
committerMatej Vasek <mvasek@redhat.com>2021-11-28 19:02:15 +0100
commitcbda62d1be5c13ac96b9fef10bc2eceead909727 (patch)
treed576d39ffba7402562ec40593381d01a4363967f /pkg/api/handlers
parent7324d94648a9987b0de2dc95cf1b6fbc20592532 (diff)
downloadpodman-cbda62d1be5c13ac96b9fef10bc2eceead909727.tar.gz
podman-cbda62d1be5c13ac96b9fef10bc2eceead909727.tar.bz2
podman-cbda62d1be5c13ac96b9fef10bc2eceead909727.zip
fix: error reporting for archive endpoint
Returning 500 when copying to read-only destination. Signed-off-by: Matej Vasek <mvasek@redhat.com>
Diffstat (limited to 'pkg/api/handlers')
-rw-r--r--pkg/api/handlers/compat/containers_archive.go4
1 files changed, 3 insertions, 1 deletions
diff --git a/pkg/api/handlers/compat/containers_archive.go b/pkg/api/handlers/compat/containers_archive.go
index cda23a399..54cbe01e9 100644
--- a/pkg/api/handlers/compat/containers_archive.go
+++ b/pkg/api/handlers/compat/containers_archive.go
@@ -133,8 +133,10 @@ func handlePut(w http.ResponseWriter, r *http.Request, decoder *schema.Decoder,
return
}
- w.WriteHeader(http.StatusOK)
if err := copyFunc(); err != nil {
logrus.Error(err.Error())
+ utils.Error(w, "Something went wrong.", http.StatusInternalServerError, err)
+ return
}
+ w.WriteHeader(http.StatusOK)
}