search

melinath / django-daguerre

On-the-fly image manipulation for Django.
http://django-daguerre.readthedocs.org/
BSD 3-Clause "New" or "Revised" License
85 stars 15 forks source link

Add X-SendFile support. #23

Closed melinath closed 11 years ago

melinath commented 12 years ago

Right now, the adjustment view redirects users to the url of the adjusted image. But some web servers (apache and nginx, for example) support the X-SendFile header on responses, which could let us save an HTTP request/response cycle. (At least for locally-stored files.)

melinath commented 11 years ago

This is obviated by the changes introduced in #22. X-Sendfile support would (even potentially) be only marginally useful, since it only works for locally-stored files, and it's actually designed as a security measure, so we'd be abusing it in any case. Now that we aren't constantly redirecting people, there's really no point in trying to implement this.