[ { "Id": "619207", "ThreadId": "259231", "Html": "\r\n

I'm trying to create and return a dotnetzip generated zip as an ActionResult in Asp.net MVC:

\r\n

 

\r\n
\r\n
\r\n
\r\n
public ActionResult DownloadZip()\r\n{\r\n\tusing (var zip = new ZipFile())\r\n\t{\r\n\t\tusing (var mstream = new MemoryStream())\r\n\t\t{\r\n\t\t\tusing(var writer = new StreamWriter(mstream, Encoding.Default))\r\n\t\t\t{\r\n\t\t\t\twriter.Write(file.ToString());\r\n\t\t\t}\r\n\t\t\tzip.AddEntry("someFile.txt", mstream);\r\n\t\t}\r\n\t\tzip.AddDirectory(Server.MapPath("~/NonWeb/someFolder"));\r\n\t\treturn new FileStreamResult(zip.GetStream(), "application/octet-stream");\r\n\t}\r\n}\r\n
\r\n
\r\n
\r\n
\r\n
However there doesn't appear to be anything like a "GetStream()" method, is there something else I can do to achieve this or does the zip file have to be written to the file system first in order to grab a stream?
\r\n",
    "PostedDate": "2011-05-27T03:02:43.763-07:00",
    "UserRole": null,
    "MarkedAsAnswerDate": null
  },
  {
    "Id": "619223",
    "ThreadId": "259231",
    "Html": "
FWIW I've tried this:
\r\n
\r\n
\r\n
public class ZipResult : ActionResult\r\n    {\r\n        private ZipFile _zipFile;\r\n        private string _filename;\r\n\r\n        public ZipResult(ZipFile zipFile, string filename)\r\n        {\r\n            _zipFile = zipFile;\r\n            _filename = filename;\r\n        }\r\n\r\n        public override void ExecuteResult(ControllerContext context)\r\n        {\r\n            if (_zipFile == null) return;\r\n\r\n            context.HttpContext.Response.ContentType = \"application/octet-stream\";\r\n            context.HttpContext.Response.AddHeader(\"content-disposition\", \"attachment; filename=\" + _filename);\r\n            // doesn't work\r\n            _zipFile.Save(context.HttpContext.Response.OutputStream);\r\n\r\n        }\r\n    }\r\n
\r\n
\r\nBut the Save method errors with an ObjectDisposedException \"Cannot access a closed Stream.\"
",
    "PostedDate": "2011-05-27T03:36:28.79-07:00",
    "UserRole": null,
    "MarkedAsAnswerDate": null
  },
  {
    "Id": "619318",
    "ThreadId": "259231",
    "Html": "
Got there in the end with the help of this stackoverflow post, for the benefit of anyone else wanting to do this:
\r\n
\r\n
\r\n
public class ZipResult : ActionResult\r\n    {\r\n        private readonly string[] _filenames;\r\n        private readonly byte[][] _files;\r\n        private readonly string _filename;\r\n\r\n        /// <summary>\r\n        /// Creates a streamed zip file for download\r\n        /// </summary>\r\n        /// <param name=\"filenames\">string array of file and foldernames to include in the zip file, folders simply end with a trailing slash (\"/\" or \"\\\")</param>\r\n        /// <param name=\"files\">byte array of files must be in same order as files in filenames array (folders are skipped)</param>\r\n        /// <param name=\"filename\">default filename for the zip file downloaded</param>\r\n        public ZipResult(string[] filenames, byte[][] files, string filename)\r\n        {\r\n            _filenames = filenames;\r\n            _files = files;\r\n            _filename = filename;\r\n        }\r\n\r\n        public override void ExecuteResult(ControllerContext context)\r\n        {\r\n            if (_files == null) return;\r\n\r\n            var response = context.HttpContext.Response;\r\n            response.Clear();\r\n            response.ClearContent();\r\n            response.ClearHeaders();\r\n            response.Cookies.Clear();\r\n            response.ContentType = \"application/octet-stream\";\r\n            response.ContentEncoding = Encoding.Default;\r\n            response.AddHeader(\"Content-Type\", \"application/octet-stream\");\r\n            response.BufferOutput = false;\r\n            context.HttpContext.Response.AddHeader(\"Content-Disposition\", String.Format(\"attachment; filename={0}\", _filename));\r\n\r\n            using (var zipOStream = new ZipOutputStream(context.HttpContext.Response.OutputStream))\r\n            {\r\n                var index = 0;\r\n                foreach(var currentFileName in _filenames)\r\n                {\r\n                    // add filename to stream\r\n                    zipOStream.PutNextEntry(currentFileName);\r\n\r\n                    // skip folders\r\n                    if(currentFileName.EndsWith(\"/\") || currentFileName.EndsWith(\"\\\\\"))\r\n                       continue;\r\n\r\n                    // write file contents\r\n                    zipOStream.Write(_files[index], 0, _files[index].Length);\r\n                    index++;\r\n                }\r\n                zipOStream.Flush();\r\n            }\r\n            response.OutputStream.Flush();\r\n        }\r\n    }\r\n
\r\n
\r\n
",
    "PostedDate": "2011-05-27T06:42:04.297-07:00",
    "UserRole": null,
    "MarkedAsAnswerDate": null
  },
  {
    "Id": "619502",
    "ThreadId": "259231",
    "Html": "
Glad you worked it out.
",
    "PostedDate": "2011-05-27T11:50:51.393-07:00",
    "UserRole": null,
    "MarkedAsAnswerDate": null
  }
]