C# MVC website PDF file in stored in byte array, display in browser

Question

I am receiving a byte[] which contains a PDF.

I need to take the byte[] and display the PDF in the browser. I have found similar questions like this - How to return PDF to browser in MVC?. But, it opens the PDF in a PDF viewer, also I am getting an error saying the file couldn't be opened because it's - "not a supported file type or because the file has been damaged".

How can I open the PDF in the browser? My code so far looks like the following -

    public ActionResult DisplayPDF()
    {
        byte[] byteArray = GetPdfFromDB();
        Stream stream = new MemoryStream(byteArray);
        stream.Flush(); 
        stream.Position = 0; 

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

Already answered here: http://stackoverflow.com/questions/779430/asp-net-mvc-how-to-get-view-to-generate-pdf — Michael Teper, Jun 05 '13 at 15:26

Weihui Guo · Answer 1 · 2020-02-23T23:45:41.583

13

If you already have the byte[], you should use FileContentResult, which "sends the contents of a binary file to the response". Only use FileStreamResult when you have a stream open.

public ActionResult DisplayPDF()
{
    byte[] byteArray = GetPdfFromDB();

    return new FileContentResult(byteArray, "application/pdf");
}

edited Feb 23 '20 at 23:45

answered Aug 16 '19 at 12:50

Weihui Guo

3,669
5
34
56

1

This worked perfect for me. Using JavaScript to open a small window and call the controller I was able to use the above code to pull out a pdf image from the database. window.open('GetInvoiceImage?ExternalImage=' + ExternalImage, 'Image_win', 'width=800,height=500,left=0,top=0,scrollbars=yes,resizable=yes,menubar=yes'); – Deathstalker Oct 17 '19 at 15:46

score 12 · Answer 2 · edited May 19 '20 at 12:22

12

You can show the byte array PDF directly in your browser simply by using MemoryStream instead of Stream and FileStreamResult instead of File:

public ActionResult DisplayPDF()
{
    byte[] byteArray = GetPdfFromDB();
    using( MemoryStream pdfStream = new MemoryStream())
    {
        pdfStream.Write(byteArray , 0,byteArray .Length);
        pdfStream.Position = 0;
        return new FileStreamResult(pdfStream, "application/pdf");
    }
}

edited May 19 '20 at 12:22

Mohsin

692
1
7
15

answered Mar 29 '16 at 16:11

Roberto C.

121
1
3

1

Hey! Put that in a using clause! – Rob Mar 17 '17 at 11:58
4

@Rob MVC disposes of streams in fileresults for you, because you cant wrap it in a using clause (the stream will be disposed before MVC gets a chance to read it) – berkeleybross Jul 23 '17 at 03:39
Use `Response.AddHeader("content-disposition", "inline;filename=Test.pdf");` to set the filename but not download the file immediately. You get `Response` for example by subclassing `System.Web.Mvc.Controller` when you have an MVC app. – bugybunny Oct 12 '18 at 14:30
inline + filename doesn’t seem to be a correct header. Chrome and Firefox respect the set username but Edge does not. So be careful and aware of that. – bugybunny Oct 12 '18 at 14:49

C# MVC website PDF file in stored in byte array, display in browser

2 Answers2

Linked