Skip to content

Commit b08edb5

Browse files
committed
Fix: fixed interpretation issue where --default now sets both resource and default permission
1 parent 0d0cff4 commit b08edb5

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

src/commands/solid-perms_acl.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ export async function changePermissions(resourceUrl: string, operations: IPermis
133133
// Update local acl for agent with new rights
134134
if (operation.default) {
135135
// remove non default entry
136-
aclDataset = setAgentResourceAccess(aclDataset, operation.id, denyAllAccess)
136+
aclDataset = setAgentResourceAccess(aclDataset, operation.id, access)
137137
// update default entry
138138
aclDataset = setAgentDefaultAccess(aclDataset, operation.id, access)
139139
} else {
@@ -152,7 +152,7 @@ export async function changePermissions(resourceUrl: string, operations: IPermis
152152
// Update local acl for group with new rights
153153
if (operation.default) {
154154
// remove non default entry
155-
aclDataset = setGroupResourceAccess(aclDataset, operation.id, denyAllAccess)
155+
aclDataset = setGroupResourceAccess(aclDataset, operation.id, access)
156156
// update default entry
157157
aclDataset = setGroupDefaultAccess(aclDataset, operation.id, access)
158158
} else {
@@ -169,7 +169,7 @@ export async function changePermissions(resourceUrl: string, operations: IPermis
169169
// Update local acl for agent with new rights
170170
if (operation.default) {
171171
// remove non default entry
172-
aclDataset = setPublicResourceAccess(aclDataset, denyAllAccess)
172+
aclDataset = setPublicResourceAccess(aclDataset, access)
173173
// update default entry
174174
aclDataset = setPublicDefaultAccess(aclDataset, access)
175175
} else {

0 commit comments

Comments
 (0)