Merge pull request #284 from cnpm/issue283-maintainer

use common.isMaintainer, fixed #283
cnpm · Mar 20, 2014 · cd9d403 · cd9d403
2 parents 2e99a77 + 6d43612
commit cd9d403
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/controllers/registry/module.js b/controllers/registry/module.js
@@ -445,10 +445,7 @@ exports.updateLatest = function *(next) {
     debug('can not get nextMod');
     return yield* next;
   }
-  var match = nextMod.package.maintainers.filter(function (item) {
-    return item.name === username;
-  });
-  if (match.length === 0) {
+  if (!common.isMaintainer(this.user, nextMod.package.maintainers)) {
     this.status = 401;
     this.body = {
       error: 'noperms',