Minor change in evaluator logic

Signed-off-by: Darshit Chanpura <dchanp@amazon.com>

Author: DarshitChanpura

src/main/java/org/opensearch/security/privileges/ResourceAccessEvaluator.java

@@ -12,6 +12,7 @@
 
 import java.util.Set;
 
+import joptsimple.internal.Strings;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
 
@@ -98,8 +99,8 @@ public boolean shouldEvaluate(ActionRequest request) {
         );
         if (!isResourceSharingFeatureEnabled) return false;
         if (!(request instanceof DocRequest docRequest)) return false;
-        if (docRequest.id() == null) {
-            log.debug("Request id is null, request is of type {}", docRequest.getClass().getName());
+        if (Strings.isNullOrEmpty(docRequest.id())) {
+            log.debug("Request id is blank or null, request is of type {}", docRequest.getClass().getName());
             return false;
         }
         // if requested index is not a resource sharing index, move on to the regular evaluator