From 5d19120cf4d938c88b8d48ad266c1950b136275d Mon Sep 17 00:00:00 2001 From: rijkvanzanten Date: Tue, 26 Apr 2022 14:34:06 -0400 Subject: [PATCH] Force share generated perm set to read --- api/src/utils/merge-permissions-for-share.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/api/src/utils/merge-permissions-for-share.ts b/api/src/utils/merge-permissions-for-share.ts index 4ff9381146..6c9504f80e 100644 --- a/api/src/utils/merge-permissions-for-share.ts +++ b/api/src/utils/merge-permissions-for-share.ts @@ -64,7 +64,9 @@ export function mergePermissionsForShare( } // Explicitly filter out permissions to collections unrelated to the root parent item. - const limitedPermissions = currentPermissions.filter(({ collection }) => allowedCollections.includes(collection)); + const limitedPermissions = currentPermissions.filter( + ({ action, collection }) => allowedCollections.includes(collection) && action === 'read' + ); return mergePermissions('and', limitedPermissions, generatedPermissions); }