(gravitino) branch main updated: [#7592] feat: add check for empty names in validateAuthorizationMetadataObject (#7650)

[jmclean]

This is an automated email from the ASF dual-hosted git repository.

jmclean pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new 62af7df4fb [#7592] feat: add check for empty names in 
validateAuthorizationMetadataObject (#7650)
62af7df4fb is described below

commit 62af7df4fb3c8c2ff7eaf8cd21ed441d18424022
Author: Reuben George <69674713+georgereu...@users.noreply.github.com>
AuthorDate: Fri Jul 11 06:13:30 2025 +0530

    [#7592] feat: add check for empty names in 
validateAuthorizationMetadataObject (#7650)
    
    ### What changes were proposed in this pull request?
    
    Added a check for empty names during validation in
    validateAuthorizationMetadataObject
    
    Fix: #7592
    
    ### Does this PR introduce any user-facing change?
    No
    
    ### How was this patch tested?
    Tested with a unit test
---
 .../apache/gravitino/authorization/common/PathBasedMetadataObject.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java
 
b/authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java
index c3bae05327..bb8362fecf 100644
--- 
a/authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java
+++ 
b/authorizations/authorization-common/src/main/java/org/apache/gravitino/authorization/common/PathBasedMetadataObject.java
@@ -155,7 +155,8 @@ public class PathBasedMetadataObject implements 
AuthorizationMetadataObject {
 
     for (String name : names) {
       Preconditions.checkArgument(
-          name != null, "Cannot create a path based metadata object with null 
name");
+          name != null && !name.isEmpty(),
+          "Cannot create a path based metadata object with null name");
     }
   }