Marmelator/persistence-service
1
0
Fork 0
Code Issues Pull Requests 17 Actions Packages Projects Releases 1.1k Wiki Activity

RED-9959: Delete and recreate acl entries that can not be created after they are not found.

Browse Source
This commit is contained in:
Dominique Eifländer 2024-08-28 09:40:18 +02:00
parent 9b88283d2f
commit f6601015be
1 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
persistence-service-v1/persistence-service-processor-v1/src/main/java/com/iqser/red/service/persistence/management/v1/processor/acl/AbstractACLService.java
View File

@ -3,12 +3,15 @@ package com.iqser.red.service.persistence.management.v1.processor.acl;
import java.io.Serializable;
import org.springframework.security.acls.domain.ObjectIdentityImpl;
import org.springframework.security.acls.model.AlreadyExistsException;
import org.springframework.security.acls.model.MutableAcl;
import org.springframework.security.acls.model.MutableAclService;
import org.springframework.security.acls.model.NotFoundException;
import lombok.RequiredArgsConstructor;
import lombok.extern.slf4j.Slf4j;
@Slf4j
@RequiredArgsConstructor
public abstract class AbstractACLService<ID extends Serializable> {
@ -21,7 +24,16 @@ public abstract class AbstractACLService<ID extends Serializable> {
try {
return (MutableAcl) mutableAclService.readAclById(objectIdentity);
} catch (NotFoundException e) {
return mutableAclService.createAcl(objectIdentity);
try {
return mutableAclService.createAcl(objectIdentity);
} catch (AlreadyExistsException e1) {
// This happened always for the same dossierId repeatedly in an endless loop triggered by SyncUserPermissionsJob every 2 mins on prod stack till it broke the stack,
// so I don't think it was any kind of race condition.
// Maybe this happens due corrupt entries.
log.warn("Recreate already existing acl object {}", objectIdentity);
mutableAclService.deleteAcl(objectIdentity, true);
return mutableAclService.createAcl(objectIdentity);
}
}
}