Skip to content

Commit 39fd741

Browse files
fix access checks when creating resources
1 parent 3da2756 commit 39fd741

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

src/services/ResourceService.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -238,9 +238,12 @@ async function init (currentUser, challengeId, resource, isCreated) {
238238

239239
// perform access validation
240240
let resources
241+
// Verify the member has agreed to the challenge terms
242+
if (isCreated) {
243+
await helper.checkAgreedTerms(memberId, _.filter(_.get(challenge, 'terms', []), t => t.roleId === resourceRole.id))
244+
}
241245
if (!currentUser.isMachine && !helper.hasAdminRole(currentUser)) {
242246
// Check if user has agreed to the challenge terms
243-
await helper.checkAgreedTerms(currentUser.userId, _.filter(_.get(challenge, 'terms', []), t => t.roleId === resourceRole.id))
244247
resources = await helper.query('Resource', { challengeId })
245248
if (!resourceRole.selfObtainable || _.toString(memberId) !== _.toString(currentUser.userId)) {
246249
// if user is not creating/deleting a self obtainable resource for itself

0 commit comments

Comments
 (0)