remove untested route set_admin
This commit is contained in:
+1
-13
@@ -5,9 +5,7 @@ exports = module.exports = {
|
|||||||
get: get,
|
get: get,
|
||||||
getAll: getAll,
|
getAll: getAll,
|
||||||
update: update,
|
update: update,
|
||||||
del: del,
|
del: del
|
||||||
|
|
||||||
setAdmin: setAdmin
|
|
||||||
};
|
};
|
||||||
|
|
||||||
var assert = require('assert'),
|
var assert = require('assert'),
|
||||||
@@ -100,13 +98,3 @@ function del(req, res, next) {
|
|||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
function setAdmin(req, res, next) {
|
|
||||||
assert.strictEqual(typeof req.params.domain, 'string');
|
|
||||||
|
|
||||||
domains.setAdmin(req.params.domain.toLowerCase(), function (error) {
|
|
||||||
if (error && error.reason === DomainError.NOT_FOUND) return next(new HttpError(404, error.message));
|
|
||||||
if (error) return next(new HttpError(500, error));
|
|
||||||
|
|
||||||
next(new HttpSuccess(202, {}));
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|||||||
@@ -251,7 +251,6 @@ function initializeExpressSync() {
|
|||||||
router.get ('/api/v1/domains', settingsScope, routes.user.requireAdmin, routes.domains.getAll);
|
router.get ('/api/v1/domains', settingsScope, routes.user.requireAdmin, routes.domains.getAll);
|
||||||
router.get ('/api/v1/domains/:domain', settingsScope, routes.user.requireAdmin, routes.domains.get);
|
router.get ('/api/v1/domains/:domain', settingsScope, routes.user.requireAdmin, routes.domains.get);
|
||||||
router.put ('/api/v1/domains/:domain', settingsScope, routes.user.requireAdmin, routes.domains.update);
|
router.put ('/api/v1/domains/:domain', settingsScope, routes.user.requireAdmin, routes.domains.update);
|
||||||
router.post('/api/v1/domains/:domain/set_admin', settingsScope, routes.user.requireAdmin, routes.domains.setAdmin);
|
|
||||||
router.del ('/api/v1/domains/:domain', settingsScope, routes.user.requireAdmin, routes.user.verifyPassword, routes.domains.del);
|
router.del ('/api/v1/domains/:domain', settingsScope, routes.user.requireAdmin, routes.user.verifyPassword, routes.domains.del);
|
||||||
|
|
||||||
// caas routes
|
// caas routes
|
||||||
|
|||||||
Reference in New Issue
Block a user