240

In a regular MVC controller, we can output pdf with a FileContentResult.

public FileContentResult Test(TestViewModel vm)
{
    var stream = new MemoryStream();
    //... add content to the stream.

    return File(stream.GetBuffer(), "application/pdf", "test.pdf");
}

But how can we change it into an ApiController?

[HttpPost]
public IHttpActionResult Test(TestViewModel vm)
{
     //...
     return Ok(pdfOutput);
}

Here is what I've tried but it doesn't seem to work.

[HttpGet]
public IHttpActionResult Test()
{
    var stream = new MemoryStream();
    //...
    var content = new StreamContent(stream);
    content.Headers.ContentType = new MediaTypeHeaderValue("application/pdf");
    content.Headers.ContentLength = stream.GetBuffer().Length;
    return Ok(content);            
}

The returned result displayed in the browser is:

{"Headers":[{"Key":"Content-Type","Value":["application/pdf"]},{"Key":"Content-Length","Value":["152844"]}]}

And there is a similar post on SO: Returning binary file from controller in ASP.NET Web API . It talks about output an existing file. But I could not make it work with a stream.

Any suggestions?

1

8 Answers 8

252

Instead of returning StreamContent as the Content, I can make it work with ByteArrayContent.

[HttpGet]
public HttpResponseMessage Generate()
{
    var stream = new MemoryStream();
    // processing the stream.

    var result = new HttpResponseMessage(HttpStatusCode.OK)
    {
        Content = new ByteArrayContent(stream.ToArray())
    };
    result.Content.Headers.ContentDisposition =
        new System.Net.Http.Headers.ContentDispositionHeaderValue("attachment")
    {
        FileName = "CertificationCard.pdf"
    };
    result.Content.Headers.ContentType =
        new MediaTypeHeaderValue("application/octet-stream");

    return result;
}
15
  • 4
    If the top half answers your question, please only post that as an answer. The second half appears to be a different question - post a new question for that.
    – gunr2171
    Dec 15, 2014 at 18:21
  • 5
    Hi, thanks for sharing, got a simple question (I guess). I have a C# front end that receives the httpresponsemessage. How do I extract the streamcontent and make it available so a user can save it to disk or something (and I can get the actual file)? Thanks!
    – Ronald
    Aug 24, 2015 at 10:06
  • 10
    I was trying to download a self generated excel file. Using the stream.GetBuffer() always returned an corrupted excel. If instead I use stream.ToArray() the file is generated without a problem. Hope this helps someone.
    – afnpires
    Apr 5, 2016 at 15:07
  • 8
    @AlexandrePires That is because MemoryStream.GetBuffer() actually returns the MemoryStream's buffer, which is usually larger than the stream's content (to make insertions efficient). MemoryStream.ToArray() returns the buffer truncated to the content size.
    – M.Stramm
    Jul 28, 2016 at 0:54
  • 42
    Please stop doing this. This sort of abuse of MemoryStream causes, unscalable code and completely ignores the purpose of Streams. Think: why isn't everything just exposed as byte[] buffers instead? Your users can easily run your application out of memory.
    – makhdumi
    Apr 6, 2017 at 23:49
118

If you want to return IHttpActionResult you can do it like this:

[HttpGet]
public IHttpActionResult Test()
{
    var stream = new MemoryStream();

    var result = new HttpResponseMessage(HttpStatusCode.OK)
    {
        Content = new ByteArrayContent(stream.GetBuffer())
    };
    result.Content.Headers.ContentDisposition = new System.Net.Http.Headers.ContentDispositionHeaderValue("attachment")
    {
        FileName = "test.pdf"
    };
    result.Content.Headers.ContentType = new MediaTypeHeaderValue("application/octet-stream");

    var response = ResponseMessage(result);

    return response;
}
4
  • 3
    Good update to show IHttpActionResult return type. A refactor of this code would be to move call a custom IHttpActionResult such as the one listed at: stackoverflow.com/questions/23768596/…
    – Josh
    May 26, 2016 at 21:19
  • This post demonstrates a nice tidy single use implementation. In my case, the helper method listed in the above link proved more helpful
    – hanzolo
    Mar 29, 2017 at 22:59
  • 1
    If you vote down please say why, hard to improve answers otherwise.
    – Ogglas
    Aug 26, 2020 at 13:53
  • I think you can only use stream.GetBuffer() if you specify the actual size using the ByteArrayContent(Byte[], Int32, Int32) constructor: new ByteArrayContent(stream.GetBuffer(), 0, checked((int)stream.Length)). The internal buffer array may be longer than the actual contents so specifying the content length is necessary.
    – dbc
    Aug 29, 2020 at 19:14
53

This question helped me.

So, try this:

Controller code:

[HttpGet]
public HttpResponseMessage Test()
{
    var path = System.Web.HttpContext.Current.Server.MapPath("~/Content/test.docx");;
    HttpResponseMessage result = new HttpResponseMessage(HttpStatusCode.OK);
    var stream = new FileStream(path, FileMode.Open);
    result.Content = new StreamContent(stream);
    result.Content.Headers.ContentDisposition = new ContentDispositionHeaderValue("attachment");
    result.Content.Headers.ContentDisposition.FileName = Path.GetFileName(path);
    result.Content.Headers.ContentType = new MediaTypeHeaderValue("application/octet-stream");
    result.Content.Headers.ContentLength = stream.Length;
    return result;          
}

View Html markup (with click event and simple url):

<script type="text/javascript">
    $(document).ready(function () {
        $("#btn").click(function () {
            // httproute = "" - using this to construct proper web api links.
            window.location.href = "@Url.Action("GetFile", "Data", new { httproute = "" })";
        });
    });
</script>


<button id="btn">
    Button text
</button>

<a href=" @Url.Action("GetFile", "Data", new { httproute = "" }) ">Data</a>
6
  • 1
    Here you are using FileStream for an existing file on the server. It is a bit different from MemoryStream. But thanks for the input.
    – Blaise
    Sep 25, 2014 at 14:26
  • 5
    If you do read from a file on a web server, be sure to use the overload for FileShare.Read, otherwise you may encounter file in use exceptions. Sep 25, 2014 at 14:34
  • if you replace it with memory stream it will not work?
    – aleha_84
    Sep 25, 2014 at 14:34
  • @JeremyBell it is just a simpified example, nobody talks here about production and fail-safe version.
    – aleha_84
    Sep 25, 2014 at 14:35
  • 1
    @Blaise See below for why this code works with FileStream but fails with MemoryStream. It's basically got to do with the Stream's Position.
    – M.Stramm
    Jul 28, 2016 at 0:49
25

Here is an implementation that streams the file's content out without buffering it (buffering in byte[] / MemoryStream, etc. can be a server problem if it's a big file).

public class FileResult : IHttpActionResult
{
    public FileResult(string filePath)
    {
        if (filePath == null)
            throw new ArgumentNullException(nameof(filePath));

        FilePath = filePath;
    }

    public string FilePath { get; }

    public Task<HttpResponseMessage> ExecuteAsync(CancellationToken cancellationToken)
    {
        var response = new HttpResponseMessage(HttpStatusCode.OK);
        response.Content = new StreamContent(File.OpenRead(FilePath));
        var contentType = MimeMapping.GetMimeMapping(Path.GetExtension(FilePath));
        response.Content.Headers.ContentType = new MediaTypeHeaderValue(contentType);
        return Task.FromResult(response);
    }
}

It can be simply used like this:

public class MyController : ApiController
{
    public IHttpActionResult Get()
    {
        string filePath = GetSomeValidFilePath();
        return new FileResult(filePath);
    }
}
3
  • 1
    How would you delete the file after the download is done? Are there any hooks to be notified when the download is finished?
    – boggy
    Nov 7, 2018 at 20:15
  • 1
    ok, the answer seems to be to implement an action filter attribute and remove the file in the OnActionExecuted method.
    – boggy
    Nov 7, 2018 at 20:45
  • 7
    Found this post Risord's answer: stackoverflow.com/questions/2041717/…. One can use this line var fs = new FileStream(FilePath, FileMode.Open, FileAccess.Read, FileShare.None, 4096, FileOptions.DeleteOnClose); instead of File.OpenRead(FilePath)
    – boggy
    Nov 7, 2018 at 22:21
8

I am not exactly sure which part to blame, but here's why MemoryStream doesn't work for you:

As you write to MemoryStream, it increments its Position property. The constructor of StreamContent takes into account the stream's current Position. So if you write to the stream, then pass it to StreamContent, the response will start from the nothingness at the end of the stream.

There's two ways to properly fix this:

  1. construct content, write to stream

     [HttpGet]
     public HttpResponseMessage Test()
     {
         var stream = new MemoryStream();
         var response = Request.CreateResponse(HttpStatusCode.OK);
         response.Content = new StreamContent(stream);
         // ...
         // stream.Write(...);
         // ...
         return response;
     }
    
  2. write to stream, reset position, construct content

     [HttpGet]
     public HttpResponseMessage Test()
     {
         var stream = new MemoryStream();
         // ...
         // stream.Write(...);
         // ...
         stream.Position = 0;
    
         var response = Request.CreateResponse(HttpStatusCode.OK);
         response.Content = new StreamContent(stream);
         return response;
     }
    
  3. looks a little better if you have a fresh Stream, 1) is simpler if your stream does not start at 0

2
  • 1
    This code actually does not provide any solution to the problem, as it uses the same approach that was mentioned int the question. The question already states that this does not work, and I can confirm that. return Ok(new StreamContent(stream)) returns JSON representation of StreamContent. Jul 27, 2016 at 19:47
  • Updated the code. This answer actually answers the more subtle question of 'why does the simple solution work with FileStream but not MemoryStream' rather than how to return a File in WebApi.
    – M.Stramm
    Jul 28, 2016 at 1:05
6

For me it was the difference between

var response = Request.CreateResponse(HttpStatusCode.OK, new StringContent(log, System.Text.Encoding.UTF8, "application/octet-stream");

and

var response = Request.CreateResponse(HttpStatusCode.OK);
response.Content = new StringContent(log, System.Text.Encoding.UTF8, "application/octet-stream");

The first one was returning the JSON representation of StringContent: {"Headers":[{"Key":"Content-Type","Value":["application/octet-stream; charset=utf-8"]}]}

While the second one was returning the file proper.

It seems that Request.CreateResponse has an overload that takes a string as the second parameter and this seems to have been what was causing the StringContent object itself to be rendered as a string, instead of the actual content.

3

I found this article useful: https://codeburst.io/download-files-using-web-api-ae1d1025f0a9

Basically it says:

[Route("api/[controller]")]
[ApiController]
public class JobController : ControllerBase
{
    [HttpGet]
    public ActionResult GetFile()
    {
        byte[] fileContent = GetFile();
        return File(fileContent, "application/pdf", "test.pdf");
    }
}
1
  • 4
    This is for .Net Core Dec 5, 2022 at 6:43
-3
 var memoryStream = new MemoryStream();
                await cloudFile.DownloadToStreamAsync(memoryStream);
                responseMessage.result = "Success";

                var contentType = "application/octet-stream";
            
                **using (var stream = new MemoryStream())
                {                    
                    return File(memoryStream.GetBuffer(), contentType, "Cartage.pdf");
                }**
1
  • 4
    Please add further details to expand on your answer, such as working code or documentation citations.
    – Community Bot
    Aug 28, 2021 at 16:18

Your Answer

Reminder: Answers generated by Artificial Intelligence tools are not allowed on Stack Overflow. Learn more

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.