From fd217b4dbfc67f72a252c7a0182413260fbd9ce0 Mon Sep 17 00:00:00 2001 From: Axel Uhl Date: Thu, 21 Jan 2021 23:14:18 +0100 Subject: [PATCH] bug4811: fixed removeSshKey regarding clean-up of ownership/access control store --- .../ui/server/LandscapeManagementWriteServiceImpl.java | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/java/com.sap.sailing.landscape.ui/src/com/sap/sailing/landscape/ui/server/LandscapeManagementWriteServiceImpl.java b/java/com.sap.sailing.landscape.ui/src/com/sap/sailing/landscape/ui/server/LandscapeManagementWriteServiceImpl.java index 9cb282debb9..7747bb6241e 100755 --- a/java/com.sap.sailing.landscape.ui/src/com/sap/sailing/landscape/ui/server/LandscapeManagementWriteServiceImpl.java +++ b/java/com.sap.sailing.landscape.ui/src/com/sap/sailing/landscape/ui/server/LandscapeManagementWriteServiceImpl.java @@ -137,10 +137,7 @@ public class LandscapeManagementWriteServiceImpl extends ResultCachingProxiedRem @Override public void removeSshKey(String awsAccessKey, String awsSecret, SSHKeyPairDTO keyPair) { - SecurityUtils.getSubject().checkPermission( - SecuredLandscapeTypes.SSH_KEY.getStringPermissionForTypeRelativeIdentifier(DefaultActions.DELETE, - keyPair.getTypeRelativeObjectIdentifier())); - AwsLandscape - .obtain(awsAccessKey, awsSecret).deleteKeyPair(new AwsRegion(keyPair.getRegionId()), keyPair.getName()); + getSecurityService().checkPermissionAndDeleteOwnershipForObjectRemoval(keyPair, + ()->AwsLandscape.obtain(awsAccessKey, awsSecret).deleteKeyPair(new AwsRegion(keyPair.getRegionId()), keyPair.getName())); } }