Configurable headers for wagtail documents

It seems Content-Disposition header is always set to ‘attachments’ https://github.com/torchbox/wagtail/blob/v0.8.7/wagtail/wagtaildocs/views/serve.py#L16 which results in browsers prompting users to download the file.

Since modern browsers can easily display PDF files, it would be great if we could make above header configurable.

[ipmb]

Ran into this on a couple of client sites, specifically related to PDF documents not being previewable.

Here’s a workaround that overrides Wagtail’s default document view and redirects to the file URL:

from django.http import HttpResponseRedirect
from django.shortcuts import get_object_or_404
from wagtail.wagtaildocs.models import Document


def serve_wagtail_doc(request, document_id, document_filename):
    """
    Replacement for ``wagtail.wagtaildocs.views.serve.serve``
    Wagtail's default document view serves everything as an attachment.
    We'll bounce back to the URL and let the media server serve it.
    """
    doc = get_object_or_404(Document, id=document_id)
    return HttpResponseRedirect(doc.file.url)

Then add this to the top of your urlpatterns (above any wagtail includes):

    # override default wagtail view
    url(r'^documents/(\d+)/(.*)$', [path.to].serve_wagtail_doc,
        name='wagtaildocs_serve'),

[eyjolfur12]

So, any news on this issue. It seams still that pdf’s inserted in RicheText or Stream Fields are served as Content-Type: application/octet-stream and Content-Disposition: attachment; and so downloaded instead of displayed in the browser. Any way yet to affect this behavior?