Merge pull request #242 from cnpm/gzip

gzip support. fix #241

package.json

@@ -29,7 +29,7 @@
     "humanize-number": "0.0.2",
     "koa": "0.5.0",
     "koa-markdown": "0.0.2",
-    "koa-middlewares": "0.0.6",
+    "koa-middlewares": "0.0.7",
     "logfilestream": "0.1.0",
     "marked": "0.3.1",
     "microtime": "0.5.1",
@@ -42,7 +42,7 @@
     "qn": "0.2.0",
     "ready": "0.1.1",
     "semver": "2.2.1",
-    "thunkify-wrap": "0.0.1",
+    "thunkify-wrap": "0.0.3",
     "urllib": "0.5.5",
     "utility": "0.1.10"
   },

servers/registry.js

@@ -40,6 +40,7 @@ app.use(middlewares.bodyParser({jsonLimit: config.jsonLimit}));
 app.use(auth());
 app.use(notFound);
 
+app.use(middlewares.gzip());
 app.use(middlewares.fresh());
 app.use(middlewares.etag());
 

servers/web.js

@@ -45,6 +45,10 @@ app.use(session);
 app.use(middlewares.bodyParser());
 app.use(notFound);
 
+app.use(middlewares.gzip());
+app.use(middlewares.fresh());
+app.use(middlewares.etag());
+
 var viewDir = path.join(rootdir, 'view', 'web');
 var docDir = path.join(rootdir, 'docs', 'web');
 

test/controllers/registry/module.test.js

@@ -86,6 +86,26 @@ describe('controllers/registry/module.test.js', function () {
       });
     });
 
+    it('should return module info and gzip when accept-encoding=gzip', function (done) {
+      request(app)
+      .get('/cnpmjs.org')
+      .set('accept-encoding', 'gzip')
+      .expect('content-encoding', 'gzip')
+      .expect(200, function (err, res) {
+        // console.log(res.headers)
+        should.not.exist(err);
+        // should have etag
+        res.headers.should.have.property('etag');
+        etag = res.headers.etag;
+        res.body.should.have.keys('_id', '_rev', 'name', 'description',
+          'versions', 'dist-tags', 'readme', 'maintainers',
+          'time', 'author', 'repository', '_attachments');
+        res.body.name.should.equal('cnpmjs.org');
+        res.body.versions[Object.keys(res.body.versions)[0]].dist.tarball.should.include('/cnpmjs.org/download');
+        done();
+      });
+    });
+
     it('should 304 when etag match', function (done) {
       request(app)
       .get('/cnpmjs.org')

test/controllers/web/package.test.js

@@ -78,9 +78,14 @@ describe('controllers/web/package.test.js', function () {
       request(app)
       .get('/package/cutter')
       .expect(200)
+      .expect('content-encoding', 'gzip')
       .expect(/<div id="package">/)
       .expect(/<th>Maintainers<\/th>/)
-      .expect(/<th>Version<\/th>/, done);
+      .expect(/<th>Version<\/th>/, function (err, res) {
+        should.not.exist(err);
+        res.should.have.header('etag');
+        done();
+      });
     });
 
     it('should get 404', function (done) {