Skip to content

Commit 27fee05

Browse files
MeritonKastratiicsm-thu
authored andcommitted
fix: [PROD-14986] inherited dataset sharing issue
1 parent ebde14c commit 27fee05

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/utils/SecurityUtils.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -99,14 +99,14 @@ const updateResourceSecurity = async (
9999
};
100100

101101
const forgeDatasetSecurityFromScenarioSecurity = (scenarioSecurity) => {
102-
const getDatasetRoleFromScenarioRole = (scenarioRole) =>
103-
scenarioRole === ACL_ROLES.SCENARIO.VALIDATOR ? ACL_ROLES.DATASET.EDITOR : scenarioRole;
102+
const datasetDefault =
103+
scenarioSecurity.default !== ACL_ROLES.SCENARIO.NONE ? ACL_ROLES.DATASET.ADMIN : ACL_ROLES.DATASET.NONE;
104104

105105
return {
106-
default: getDatasetRoleFromScenarioRole(scenarioSecurity.default),
106+
default: datasetDefault,
107107
accessControlList: scenarioSecurity.accessControlList.map((aclEntry) => ({
108108
id: aclEntry.id,
109-
role: getDatasetRoleFromScenarioRole(aclEntry.role),
109+
role: ACL_ROLES.DATASET.ADMIN,
110110
})),
111111
};
112112
};

0 commit comments

Comments
 (0)