Adds getResourceName as API contract for Resource

Signed-off-by: Darshit Chanpura <[email protected]>
DarshitChanpura · Dec 20, 2024 · f90b165 · f90b165
1 parent 1aff350
commit f90b165
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 4 deletions.
diff --git a/...usterTest/java/org/opensearch/accesscontrol/resources/testplugins/TestResourcePlugin.java b/...usterTest/java/org/opensearch/accesscontrol/resources/testplugins/TestResourcePlugin.java
@@ -101,13 +101,16 @@ public String getWriteableName() {
         }
 
         @Override
-        public void writeTo(StreamOutput out) throws IOException {
-
-        }
+        public void writeTo(StreamOutput out) {}
 
         @Override
         public XContentBuilder toXContent(XContentBuilder builder, Params params) throws IOException {
             return null;
         }
+
+        @Override
+        public String getResourceName() {
+            return name;
+        }
     }
 }
diff --git a/server/src/main/java/org/opensearch/accesscontrol/resources/Resource.java b/server/src/main/java/org/opensearch/accesscontrol/resources/Resource.java
@@ -14,4 +14,12 @@
 /**
  * Marker interface for all resources
  */
-public interface Resource extends NamedWriteable, ToXContentFragment {}
+public interface Resource extends NamedWriteable, ToXContentFragment {
+    /**
+     * Get the resource name
+     * @return resource name
+     */
+    String getResourceName();
+
+    // TODO: Next iteration, check if getResourceType() should be implemented
+}