This is an automated email from the ASF dual-hosted git repository.
bharat pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ozone.git
The following commit(s) were added to refs/heads/master by this push:
new f153236 HDDS-5863. Error message having null fields on volume
creation (#2752)
f153236 is described below
commit f153236982fc40824c2ceaba370db30c03c9b3c5
Author: Aswin Shakil Balasubramanian <[email protected]>
AuthorDate: Fri Oct 29 12:52:36 2021 -0700
HDDS-5863. Error message having null fields on volume creation (#2752)
---
.../java/org/apache/hadoop/ozone/om/OzoneManager.java | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git
a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java
b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java
index 7e49ab5..0750b30 100644
---
a/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java
+++
b/hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java
@@ -2098,16 +2098,19 @@ public final class OzoneManager extends
ServiceRuntimeInfoImpl
if (!accessAuthorizer.checkAccess(obj, context)) {
if (throwIfPermissionDenied) {
- LOG.warn("User {} doesn't have {} permission to access {} /{}/{}/{}",
+ String volumeName = obj.getVolumeName() != null?
+ "Volume:" + obj.getVolumeName() + " ": "";
+ String bucketName = obj.getBucketName() != null?
+ "Bucket:" + obj.getBucketName() + " ": "";
+ String keyName = obj.getKeyName() != null?
+ "Key:" + obj.getKeyName() : "";
+ LOG.warn("User {} doesn't have {} permission to access {} {}{}{}",
context.getClientUgi().getUserName(), context.getAclRights(),
- obj.getResourceType(), obj.getVolumeName(), obj.getBucketName(),
- obj.getKeyName());
+ obj.getResourceType(), volumeName, bucketName, keyName);
throw new OMException("User " + context.getClientUgi().getUserName() +
" doesn't have " + context.getAclRights() +
" permission to access " + obj.getResourceType() + " " +
- obj.getVolumeName() + " " + obj.getBucketName() + " " +
- obj.getKeyName(),
- ResultCodes.PERMISSION_DENIED);
+ volumeName + bucketName + keyName, ResultCodes.PERMISSION_DENIED);
}
return false;
} else {
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
