Merge pull request #97 from strongloop/bug/check-access
Dont attempt access checking on models without a check access method
This commit is contained in:
commit
591ed86444
|
@ -164,23 +164,27 @@ app.enableAuth = function() {
|
||||||
var modelInstance = ctx.instance;
|
var modelInstance = ctx.instance;
|
||||||
var modelId = modelInstance && modelInstance.id || req.param('id');
|
var modelId = modelInstance && modelInstance.id || req.param('id');
|
||||||
|
|
||||||
Model.checkAccess(
|
if(Model.checkAccess) {
|
||||||
req.accessToken,
|
Model.checkAccess(
|
||||||
modelId,
|
req.accessToken,
|
||||||
method.name,
|
modelId,
|
||||||
function(err, allowed) {
|
method.name,
|
||||||
if(err) {
|
function(err, allowed) {
|
||||||
console.log(err);
|
if(err) {
|
||||||
next(err);
|
console.log(err);
|
||||||
} else if(allowed) {
|
next(err);
|
||||||
next();
|
} else if(allowed) {
|
||||||
} else {
|
next();
|
||||||
var e = new Error('Access Denied');
|
} else {
|
||||||
e.statusCode = 401;
|
var e = new Error('Access Denied');
|
||||||
next(e);
|
e.statusCode = 401;
|
||||||
|
next(e);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
);
|
||||||
);
|
} else {
|
||||||
|
next();
|
||||||
|
}
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue