Created
April 14, 2021 18:16
-
-
Save jasondlee/709bc4b2e07cacab3cc8a539624da9d4 to your computer and use it in GitHub Desktop.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
diff --git a/domain-management/src/main/java/org/jboss/as/domain/management/access/RoleMappingRemove.java b/domain-management/src/main/java/org/jboss/as/domain/management/access/RoleMappingRemove.java | |
index e5cafc6758..e744361fc1 100644 | |
--- a/domain-management/src/main/java/org/jboss/as/domain/management/access/RoleMappingRemove.java | |
+++ b/domain-management/src/main/java/org/jboss/as/domain/management/access/RoleMappingRemove.java | |
@@ -33,6 +33,8 @@ import org.jboss.as.controller.OperationFailedException; | |
import org.jboss.as.controller.OperationStepHandler; | |
import org.jboss.as.controller.PathAddress; | |
import org.jboss.as.controller.access.management.WritableAuthorizerConfiguration; | |
+import org.jboss.as.controller.logging.ControllerLogger; | |
+import org.jboss.as.controller.registry.Resource; | |
import org.jboss.as.domain.management.logging.DomainManagementLogger; | |
import org.jboss.dmr.ModelNode; | |
@@ -57,14 +59,19 @@ public class RoleMappingRemove implements OperationStepHandler { | |
@Override | |
public void execute(OperationContext context, ModelNode operation) throws OperationFailedException { | |
- context.removeResource(PathAddress.EMPTY_ADDRESS); | |
+ final Resource resource = context.removeResource(PathAddress.EMPTY_ADDRESS); | |
+ if (resource == null) { | |
+ ControllerLogger.ROOT_LOGGER.managementResourceNotFound(context.getCurrentAddress()); | |
+ ControllerLogger.ROOT_LOGGER.managementResourceNotFound(context.getCurrentAddress()); | |
+ } else { | |
- PathAddress address = PathAddress.pathAddress(operation.get(OP_ADDR)); | |
- final String roleName = address.getLastElement().getValue(); | |
+ PathAddress address = PathAddress.pathAddress(operation.get(OP_ADDR)); | |
+ final String roleName = address.getLastElement().getValue(); | |
- RbacSanityCheckOperation.addOperation(context); | |
+ RbacSanityCheckOperation.addOperation(context); | |
- registerRuntimeRemove(context, roleName.toUpperCase(Locale.ENGLISH)); | |
+ registerRuntimeRemove(context, roleName.toUpperCase(Locale.ENGLISH)); | |
+ } | |
} | |
private void registerRuntimeRemove(final OperationContext context, final String roleName) { |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment