diff --git a/server/src/main/java/org/opensearch/accesscontrol/resources/ShareWith.java b/server/src/main/java/org/opensearch/accesscontrol/resources/ShareWith.java
index d474d7cb762da..1761efb45af3e 100644
--- a/server/src/main/java/org/opensearch/accesscontrol/resources/ShareWith.java
+++ b/server/src/main/java/org/opensearch/accesscontrol/resources/ShareWith.java
@@ -67,7 +67,6 @@ public XContentBuilder toXContent(XContentBuilder builder, Params params) throws
     public static ShareWith fromXContent(XContentParser parser) throws IOException {
         List<SharedWithScope> sharedWithScopes = new ArrayList<>();
 
-        // Ensure we're at the start of the object
         if (parser.currentToken() != XContentParser.Token.START_OBJECT) {
             parser.nextToken();
         }
diff --git a/server/src/main/java/org/opensearch/plugins/NoOpResourceAccessControlPlugin.java b/server/src/main/java/org/opensearch/plugins/NoOpResourceAccessControlPlugin.java
index ad67ecc6e9ef1..07c3e45c6f89a 100644
--- a/server/src/main/java/org/opensearch/plugins/NoOpResourceAccessControlPlugin.java
+++ b/server/src/main/java/org/opensearch/plugins/NoOpResourceAccessControlPlugin.java
@@ -31,6 +31,7 @@ public class NoOpResourceAccessControlPlugin implements ResourceAccessControlPlu
     @Override
     public List<String> listAccessibleResourcesInPlugin(String systemIndexName) {
         // returns an empty list since security plugin is disabled
+        // TODO: check whether this should return all entries in the given index
         return List.of();
     }