on master help sign in
Got rid of gzip/gzip' divide
commit6fcaa104173b08de232bbb800b4210c97ade35de authorMichael Snoyman <michael@snoyman.com> dateFri 6 Jan 2012, 13:38 (15 years ago) parentbc89366
2 files changed+3−10
wai-extra/Network/Wai/Middleware/Gzip.hs+2−9 wai-extra/test/WaiExtraTest.hs+1−1
wai-extra/Network/Wai/Middleware/Gzip.hs+2−9
@@ -16,7 +16,6 @@
1616---------------------------------------------------------
1717module Network.Wai.Middleware.Gzip
1818 ( gzip
19− , gzip'
2019 , GzipSettings
2120 , gzipFiles
2221 , GzipFiles (..)
@@ -62,14 +61,8 @@ defaultCheckMime = S8.isPrefixOf "text/"
6261-- Possible future enhancements:
6362--
6463-- * Only compress if the response is above a certain size.
65−gzip :: Bool -- ^ should we gzip files?
66− -> Middleware
67−gzip files = gzip' def
68− { gzipFiles = if files then GzipCompress else GzipIgnore
69− }
70−
71−gzip' :: GzipSettings -> Middleware
72−gzip' set app env = do
64+gzip :: GzipSettings -> Middleware
65+gzip set app env = do
7366 res <- app env
7467 case res of
7568 ResponseFile{} | gzipFiles set == GzipIgnore -> return res
wai-extra/test/WaiExtraTest.hs+1−1
@@ -261,7 +261,7 @@ caseJsonp = flip runSession jsonpApp $ do
261261 assertBody "{\"foo\":\"bar\"}" sres3
262262
263263gzipApp :: Application
264−gzipApp = gzip True $ const $ return $ responseLBS status200
264+gzipApp = gzip def $ const $ return $ responseLBS status200
265265 [("Content-Type", "text/plain")]
266266 "test"
267267