Skip to content
Browse files

Fixed #4946 -- Added some small improvements to Gzip middleware. Than…


git-svn-id: bcc190cf-cafb-0310-a4f2-bffc1f526a37
  • Loading branch information...
1 parent 3757f30 commit c050b6a25acff020fd60f2f528eb019615e00203 @malcolmt malcolmt committed
Showing with 11 additions and 0 deletions.
  1. +5 −0 django/middleware/
  2. +6 −0 docs/middleware.txt
5 django/middleware/
@@ -11,6 +11,11 @@ class GZipMiddleware(object):
on the Accept-Encoding header.
def process_response(self, request, response):
+ if response.status_code != 200 or len(response.content) < 200:
+ # Not worth compressing really short responses or 304 status
+ # responses, etc.
+ return response
patch_vary_headers(response, ('Accept-Encoding',))
# Avoid gzipping if we've already got a content-encoding or if the
6 docs/middleware.txt
@@ -91,6 +91,12 @@ django.middleware.gzip.GZipMiddleware
Compresses content for browsers that understand gzip compression (all modern
+It is suggested to place this first in the middleware list, so that the
+compression of the response content is the last thing that happens. Will not
+compress content bodies less than 200 bytes long, when the response code is
+something other than 200, Javascript files (for IE compatibitility), or
+responses that have the ``Content-Encoding`` header already specified.

0 comments on commit c050b6a

Please sign in to comment.
Something went wrong with that request. Please try again.