Refactor download method using SkipResponseBodyEncodeDecode

internal/api/api.go

@@ -67,8 +67,7 @@ func HTTPServer(
 	var collectionEndpoints *collection.Endpoints = collection.NewEndpoints(colsvc.Goa())
 	collectionErrorHandler := errorHandler(logger, "Collection error.")
 	var collectionServer *collectionsvr.Server = collectionsvr.New(collectionEndpoints, mux, dec, enc, collectionErrorHandler, nil, websocketUpgrader, nil)
-	// Intercept request in Download endpoint so we can serve the file directly.
-	collectionServer.Download = writeTimeout(colsvc.HTTPDownload(mux, dec), 0)
+	collectionServer.Download = writeTimeout(collectionServer.Download, 0)
 	collectionsvr.Mount(mux, collectionServer)
 
 	// Swagger service.

internal/api/design/collection.go

@@ -135,11 +135,21 @@ var _ = Service("collection", func() {
 			Attribute("id", UInt, "Identifier of collection to look up")
 			Required("id")
 		})
-		Result(Bytes)
+		Result(func() {
+			Attribute("content_type", String)
+			Attribute("content_length", Int64)
+			Attribute("content_disposition", String)
+			Required("content_type", "content_length", "content_disposition")
+		})
 		Error("not_found", CollectionNotFound, "Collection not found")
 		HTTP(func() {
 			GET("/{id}/download")
-			Response(StatusOK)
+			SkipResponseBodyEncodeDecode()
+			Response(func() {
+				Header("content_type:Content-Type")
+				Header("content_length:Content-Length")
+				Header("content_disposition:Content-Disposition")
+			})
 			Response("not_found", StatusNotFound)
 		})
 	})