Rename to isSharedWithCurrentUser

Signed-off-by: Craig Perkins <[email protected]>
cwperks · Dec 23, 2024 · e055847 · e055847
1 parent e280cf2
commit e055847
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/spi/src/main/java/org/opensearch/security/spi/DefaultResourceSharingService.java b/spi/src/main/java/org/opensearch/security/spi/DefaultResourceSharingService.java
@@ -5,7 +5,7 @@
 public class DefaultResourceSharingService<T extends Resource> implements ResourceSharingService<T> {
 
     @Override
-    public void hasResourceBeenSharedWith(String resourceId, ActionListener<Boolean> resourceSharingListener) {
+    public void isSharedWithCurrentUser(String resourceId, ActionListener<Boolean> resourceSharingListener) {
         resourceSharingListener.onResponse(Boolean.TRUE);
     }
 }
diff --git a/spi/src/main/java/org/opensearch/security/spi/ResourceSharingService.java b/spi/src/main/java/org/opensearch/security/spi/ResourceSharingService.java
@@ -3,5 +3,5 @@
 import org.opensearch.core.action.ActionListener;
 
 public interface ResourceSharingService<T extends Resource> {
-    void hasResourceBeenSharedWith(String resourceId, ActionListener<Boolean> resourceSharingListener);
+    void isSharedWithCurrentUser(String resourceId, ActionListener<Boolean> resourceSharingListener);
 }
diff --git a/...ain/java/org/opensearch/security/spi/actions/resource/get/GetResourceTransportAction.java b/...ain/java/org/opensearch/security/spi/actions/resource/get/GetResourceTransportAction.java
@@ -111,7 +111,7 @@ public void onFailure(Exception e) {
                             }
                         };
 
-                        resourceSharingService.hasResourceBeenSharedWith(request.getResourceId(), shareListener);
+                        resourceSharingService.isSharedWithCurrentUser(request.getResourceId(), shareListener);
                     } catch (IOException e) {
                         throw new OpenSearchException("Caught exception while loading resources: " + e.getMessage());
                     }

diff --git a/...n/java/org/opensearch/security/spi/actions/resource/list/ListResourceTransportAction.java b/...n/java/org/opensearch/security/spi/actions/resource/list/ListResourceTransportAction.java
@@ -123,7 +123,7 @@ public void onFailure(Exception e) {
                                 }
                             };
 
-                            resourceSharingService.hasResourceBeenSharedWith(hit.getId(), shareListener);
+                            resourceSharingService.isSharedWithCurrentUser(hit.getId(), shareListener);
 
                         } catch (IOException e) {
                             listResourceListener.onFailure(

diff --git a/src/main/java/org/opensearch/security/resource/SecurityResourceSharingService.java b/src/main/java/org/opensearch/security/resource/SecurityResourceSharingService.java
@@ -81,7 +81,7 @@ private boolean hasPermissionsFor(User authenticatedUser, ResourceSharingEntry s
     }
 
     @Override
-    public void hasResourceBeenSharedWith(String resourceId, ActionListener<Boolean> resourceSharingListener) {
+    public void isSharedWithCurrentUser(String resourceId, ActionListener<Boolean> resourceSharingListener) {
         User authenticatedUser = (User) client.threadPool()
             .getThreadContext()
             .getPersistent(ConfigConstants.OPENDISTRO_SECURITY_AUTHENTICATED_USER);