verdnatura-mirror
/
loopback
mirror of https://github.com/strongloop/loopback.git
0
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #680 from arlaneenalra/feature/fix-676 

Force principalId to be a string in Role.getRoles Fix #676
This commit is contained in:
Raymond Feng 2014-10-23 08:55:17 -07:00
parent 94033312e6 f678107859
commit 03b3c3cda4
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
common/models/role.js
View File

@ -353,7 +353,11 @@ module.exports = function(Role) {
context.principals.forEach(function(p) {
// Check against the role mappings
var principalType = p.type || undefined;
var principalId = p.id || undefined;
var principalId = p.id == null ? undefined : p.id;
if(typeof principalId !== 'string' && principalId != null) {
principalId = principalId.toString();
}
// Add the role itself
if (principalType === RoleMapping.ROLE && principalId) {