From aba191def230eab53cc68683400c307711d536e8 2016-11-02 20:47:13
From: Brett Smith <brett@sfconservancy.org>
Date: 2016-11-02 20:47:13
Subject: [PATCH] static: Don't try to render non-template files as templates.

---

diff --git a/www/conservancy/static/views.py b/www/conservancy/static/views.py
index 405b3973bab554ab4873896c2afaee9eb78faa9e..9df58c60f72d868a39cd18fe72231eb8483ce5ef 100644
--- a/www/conservancy/static/views.py
+++ b/www/conservancy/static/views.py
@@ -1,3 +1,4 @@
+import mimetypes
 import os.path
 from django.http import HttpResponse
 from django.template import RequestContext, loader
@@ -35,7 +36,6 @@ def fundgoal_lookup(fundraiser_sought):
         return None
 
 def index(request, *args, **kwargs):
-    # return HttpResponse("Hello, static world: " + request.get_full_path())
     path = request.path
     try:
         path_bytes = path.encode(FILESYSTEM_ENCODING)
@@ -45,18 +45,23 @@ def index(request, *args, **kwargs):
         path += u'index.html'
     fullpath = os.path.join(STATIC_ROOT, path_bytes)
     if not os.path.exists(fullpath):
-        # return HttpResponse("Sorry that's a 404: " + path)
         return handler404(request)
-    template = loader.get_template(path)
+    content_type, _ = mimetypes.guess_type(path)
+    if content_type != 'text/html':
+        content = open(fullpath)
+    else:
+        content_type = None  # Let Django use its default
+        template = loader.get_template(path)
 
-    kwargs = kwargs.copy()
-    if kwargs.has_key('fundraiser_sought'):
-        kwargs['fundgoal'] = fundgoal_lookup(kwargs['fundraiser_sought'])
+        kwargs = kwargs.copy()
+        if kwargs.has_key('fundraiser_sought'):
+            kwargs['fundgoal'] = fundgoal_lookup(kwargs['fundraiser_sought'])
 
-    kwargs['sitefundgoal'] = fundgoal_lookup('supporterrun')
+        kwargs['sitefundgoal'] = fundgoal_lookup('supporterrun')
 
-    context = RequestContext(request, kwargs)
-    return HttpResponse(template.render(context))
+        context = RequestContext(request, kwargs)
+        content = template.render(context)
+    return HttpResponse(content, content_type)
 
 def debug(request):
     path = request.get_full_path()