diff options
| author | Matej Vasek <mvasek@redhat.com> | 2021-11-28 19:02:15 +0100 |
|---|---|---|
| committer | Matej Vasek <mvasek@redhat.com> | 2021-11-28 19:02:15 +0100 |
| commit | cbda62d1be5c13ac96b9fef10bc2eceead909727 (patch) | |
| tree | d576d39ffba7402562ec40593381d01a4363967f /pkg/api | |
| parent | 7324d94648a9987b0de2dc95cf1b6fbc20592532 (diff) | |
| download | podman-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')
| -rw-r--r-- | pkg/api/handlers/compat/containers_archive.go | 4 |
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) } |