better error handling

cernbox · Nov 18, 2022 · 2f2cdc8 · 2f2cdc8
1 parent 8678c14
commit 2f2cdc8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/internal/http/services/owncloud/ocdav/download.go b/internal/http/services/owncloud/ocdav/download.go
@@ -78,6 +78,8 @@ func (s *svc) handleHttpError(w http.ResponseWriter, err error, log *zerolog.Log
 	switch err.(type) {
 	case errtypes.NotFound:
 		http.Error(w, "Resource not found", http.StatusNotFound)
+	case manager.ErrMaxSize, manager.ErrMaxFileCount:
+		http.Error(w, err.Error(), http.StatusRequestEntityTooLarge)
 	default:
 		http.Error(w, err.Error(), http.StatusInternalServerError)
 	}
@@ -215,8 +217,6 @@ func (s *svc) downloadArchive(ctx context.Context, w http.ResponseWriter, token
 		return
 	}
 
-	w.WriteHeader(http.StatusOK)
-
 	if err := archiver.CreateTar(ctx, w); err != nil {
 		s.handleHttpError(w, err, log)
 		return