Adding legacy fix and test for #14

tomgco · Dec 21, 2011 · 17bacdc · 17bacdc
1 parent bffc5a0
commit 17bacdc
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/lib/staticGzip.js b/lib/staticGzip.js
@@ -150,19 +150,19 @@ exports = module.exports = function staticGzip(dirPath, options){
 		contentType = contentType + (charset ? '; charset=' + charset : '');
 		acceptEncoding = req.headers['accept-encoding'] || '';
 
-		if (!contentTypeMatch.test(contentType)) {
-			return pass(filename);
-		}
-
-		if (!~acceptEncoding.indexOf('gzip')) {
-			return pass(filename);
-		}
-
 		//This is storing in memory for the moment, need to think what the best way to do this.
 		//Check file is not a directory
 
 		fs.stat(filename, function(err, stat) {
 			if (err || stat.isDirectory()) {
+				return pass(req.url);
+			}
+
+			if (!contentTypeMatch.test(contentType)) {
+				return pass(filename);
+			}
+
+			if (!~acceptEncoding.indexOf('gzip')) {
 				return pass(filename);
 			}
 

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
 "name" : "gzippo",
-"version" : "0.0.7",
+"version" : "0.0.8",
 "author" : "Tom Gallacher",
 "description" : "Gzip middleware for Connect using Compress for better performace.",
 "homepage" : "http://www.tomg.co/gzippo",

diff --git a/test/fixtures/js/nestedTest.js b/test/fixtures/js/nestedTest.js
diff --git a/test/staticGzipTest.js b/test/staticGzipTest.js
@@ -159,7 +159,7 @@ module.exports = {
 		);
 	},
 	'Ensuring that when viewing a directory a redirect works correctly': function() {
-		assert.response(getApp(),
+		assert.response(app,
 			{
 				url: '/js'
 			},