Recommended way to create an ActionResult with a file extension

Question

I need to create an ActionResult in an ASP.NET MVC application which has a .csv filetype.

I will provide a 'do not call' email list to my marketing partners and i want it to have a .csv extension in the filetype. Then it'll automatically open in Excel.

 http://www.example.com/mailinglist/donotemaillist.csv?password=12334

I have successfully done this as follows, but I want to make sure this is the absolute best and recommended way of doing this.

    [ActionName("DoNotEmailList.csv")]
    public ContentResult DoNotEmailList(string username, string password)
    {
            return new ContentResult()
            {
                Content = Emails.Aggregate((a,b)=>a+Environment.NewLine + b), 
                ContentType = "text/csv"
            };
    }

This Actionmethod will respond to the above link just fine.

I'm just wondering if there is any likelihood of any unexpected conflict of having the file extension like this with any different version of IIS, any kind of ISAPI filter, or anything else I cant think of now.

I need to be 100% sure because I will be providing this to external partners and don't want to have to change my mind later. I really cant see any issues, but maybe theres something obscure - or another more "MVC" like way of doing this.

score 30 · Answer 1 · edited Nov 14 '18 at 13:00

30

I used the FileContentResult action to also do something similar.

public FileContentResult DoNotEmailList(string username, string password)
{
        string csv = Emails.Aggregate((a,b)=>a+Environment.NewLine + b);
        byte[] csvBytes = ASCIIEncoding.ASCII.GetBytes( csv );
        return File(csvBytes, "text/csv", "DoNotEmailList.csv");
}

It will add the content-disposition header for you.

edited Nov 14 '18 at 13:00

Reinstate Monica

2,767
3
31
40

answered Oct 19 '11 at 12:55

Peter Riesz

3,091
29
33

1

Makes much more sense than writing a class to do something that's already built in! Cheers! – Shawson Apr 03 '12 at 14:39
Thanks for a simple clean answer that is easy to implement in another situation – Mattias Åslund Mar 12 '13 at 06:57

score 22 · Accepted Answer · edited Feb 02 '13 at 20:12

22

I think your Response MUST contain "Content-Disposition" header in this case. Create custom ActionResult like this:

public class MyCsvResult : ActionResult {

    public string Content {
        get;
        set;
    }

    public Encoding ContentEncoding {
        get;
        set;
    }

    public string Name {
        get;
        set;
    }

    public override void ExecuteResult(ControllerContext context) {
        if (context == null) {
            throw new ArgumentNullException("context");
        }

        HttpResponseBase response = context.HttpContext.Response;

        response.ContentType = "text/csv";

        if (ContentEncoding != null) {
            response.ContentEncoding = ContentEncoding;
        }

        var fileName = "file.csv";

        if(!String.IsNullOrEmpty(Name)) {
            fileName = Name.Contains('.') ? Name : Name + ".csv";
        }

        response.AddHeader("Content-Disposition",
            String.Format("attachment; filename={0}", fileName));

        if (Content != null) {
            response.Write(Content);
        }
    }
}

And use it in your Action instead of ContentResult:

return new MyCsvResult {
    Content = Emails.Aggregate((a,b) => a + Environment.NewLine + b)
    /* Optional
     * , ContentEncoding = ""
     * , Name = "DoNotEmailList.csv"
     */
};

edited Feb 02 '13 at 20:12

Dylan Beattie

53,688
35
128
197

answered Jun 13 '09 at 12:19

eu-ge-ne

28,023
6
71
62

thanks! but can you clarify what you mean by MUST ? what i had does appear to work - although this way is much more preferable – Simon_Weaver Jun 13 '09 at 17:41
If your Action returns ".csv" file then your clients want to "open" it or "save" it - so you must provide "Content-Disposition" header (although it is optional, see http://www.ietf.org/rfc/rfc2183.txt). Without this header the process of opening your file may differ in various browsers/OSes/machines – eu-ge-ne Jun 13 '09 at 18:06
1

Is there an error in the code? Shouldn't Name be a string instead of encoding? Also the compiler is telling me that ActionResult doesn't have a Response property – ADB Feb 07 '10 at 14:51
Thanks, AD - `Name` property fixed. `ActionResult` does not have `Response` property, but `context.HttpContext` has one. – eu-ge-ne Feb 07 '10 at 16:38
2

I've found the bug (`Response.Headers.Add ...`) - fixed (should be `response.Headers.Add ...`) - thanks again AD. – eu-ge-ne Feb 07 '10 at 16:42
MVC2 ASP.NET 4.0: Use response.AppendHeader instead of response.Headers.Add. – mxmissile May 13 '10 at 00:19
5

Got an error when running this code (MVC2, ASP.Net 3.5): "This operation requires IIS integrated pipeline mode". Looks like need to change response.Headers.Add to response.AddHeader - http://www.platinumbay.com/blogs/dotneticated/archive/2010/10/07/this-operation-requires-iis-integrated-pipeline-mode.aspx – Andy Butland Jan 12 '11 at 15:43
Thanks, useful improvement over the standard ContentResult – Paul Hadfield Jun 17 '11 at 15:38

Talljoe · Answer 3 · 2009-06-13T04:09:28.583

6

This is how I'm doing something similar. I'm treating it as a download:

var disposition = String.Format(
  "attachment;filename=\"{0}.csv\"", this.Model.Name);
Response.AddHeader("content-disposition", disposition);

This should show up in the browser as a file download with the given filename.

I can't think of a reason why yours wouldn't work, though.

edited Jun 13 '09 at 04:09

answered Jun 13 '09 at 04:03

Talljoe

14,593
4
43
39

score 0 · Answer 4 · edited May 23 '17 at 12:32

The answer you accepted is good enough, but it keeps the content of the output in memory as it outputs it. What if the file it generates is rather large? For example, when you dump a contents of the SQL table. Your application could run out of memory. What you do want in this case is to use FileStreamResult. One way to feed the data into the stream could be using pipe, as I described here

Recommended way to create an ActionResult with a file extension

4 Answers4

Linked