Browse Source

Merge pull request #1207 from Budibase/bug/perms-fix

Permissions fix - fetch enriched was 403ing in public
pull/4023/head
Michael Drury 6 years ago
committed by GitHub
parent
commit
c5e882b218
  1. 4
      packages/server/src/utilities/security/permissions.js

4
packages/server/src/utilities/security/permissions.js

@ -138,7 +138,7 @@ exports.doesHaveResourcePermission = (
) => { ) => {
// set foundSub to not subResourceId, incase there is no subResource // set foundSub to not subResourceId, incase there is no subResource
let foundMain = false, let foundMain = false,
foundSub = !subResourceId foundSub = false
for (let [resource, level] of Object.entries(permissions)) { for (let [resource, level] of Object.entries(permissions)) {
const levels = getAllowedLevels(level) const levels = getAllowedLevels(level)
if (resource === resourceId && levels.indexOf(permLevel) !== -1) { if (resource === resourceId && levels.indexOf(permLevel) !== -1) {
@ -156,7 +156,7 @@ exports.doesHaveResourcePermission = (
break break
} }
} }
return foundMain && foundSub return foundMain || foundSub
} }
exports.doesHaveBasePermission = (permType, permLevel, permissionIds) => { exports.doesHaveBasePermission = (permType, permLevel, permissionIds) => {

Loading…
Cancel
Save