feat(archive): output status 400 if archive not enabled

Author: marjune163

src/serverHandler/archiveTar.go

@@ -48,6 +48,11 @@ func writeTar(tw *tar.Writer, f *os.File, fInfo os.FileInfo, archivePath string)
 }
 
 func (h *handler) tar(w http.ResponseWriter, r *http.Request, pageData *responseData) {
+	if !pageData.CanArchive {
+		w.WriteHeader(http.StatusBadRequest)
+		return
+	}
+
 	selections, ok := h.getArchiveSelections(r)
 	if !ok {
 		return
@@ -73,6 +78,11 @@ func (h *handler) tar(w http.ResponseWriter, r *http.Request, pageData *response
 }
 
 func (h *handler) tgz(w http.ResponseWriter, r *http.Request, pageData *responseData) {
+	if !pageData.CanArchive {
+		w.WriteHeader(http.StatusBadRequest)
+		return
+	}
+
 	selections, ok := h.getArchiveSelections(r)
 	if !ok {
 		return

src/serverHandler/archiveZip.go

@@ -38,6 +38,11 @@ func writeZip(zw *zip.Writer, f *os.File, fInfo os.FileInfo, archivePath string)
 }
 
 func (h *handler) zip(w http.ResponseWriter, r *http.Request, pageData *responseData) {
+	if !pageData.CanArchive {
+		w.WriteHeader(http.StatusBadRequest)
+		return
+	}
+
 	selections, ok := h.getArchiveSelections(r)
 	if !ok {
 		return

src/serverHandler/main.go

@@ -109,23 +109,16 @@ func (h *handler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 	}
 
 	// regular flows
-
 	if len(r.URL.RawQuery) >= 3 {
 		switch r.URL.RawQuery[:3] {
 		case "tar":
-			if data.CanArchive {
-				h.tar(w, r, data)
-			}
+			h.tar(w, r, data)
 			return
 		case "tgz":
-			if data.CanArchive {
-				h.tgz(w, r, data)
-			}
+			h.tgz(w, r, data)
 			return
 		case "zip":
-			if data.CanArchive {
-				h.zip(w, r, data)
-			}
+			h.zip(w, r, data)
 			return
 		}
 	}