Remove subject authenticate method as well

Signed-off-by: Stephen Crawford <[email protected]>
stephen-crawford · Oct 13, 2023 · bba635e · bba635e
1 parent 5132a52
commit bba635e
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 27 deletions.
diff --git a/plugins/identity-shiro/src/main/java/org/opensearch/identity/shiro/ShiroSubject.java b/plugins/identity-shiro/src/main/java/org/opensearch/identity/shiro/ShiroSubject.java
@@ -78,14 +78,4 @@ public int hashCode() {
     public String toString() {
         return "ShiroSubject(principal=" + getPrincipal() + ")";
     }
-
-    /**
-     * Logs the user in via authenticating the user against current Shiro realm
-     * @param authenticationToken The authToken to be used for login
-     */
-    public void authenticate(AuthToken authenticationToken) {
-        final org.apache.shiro.authc.AuthenticationToken authToken = authTokenHandler.translateAuthToken(authenticationToken)
-            .orElseThrow(() -> new UnsupportedAuthenticationToken());
-        shiroSubject.login(authToken);
-    }
 }
diff --git a/server/src/main/java/org/opensearch/identity/Subject.java b/server/src/main/java/org/opensearch/identity/Subject.java
@@ -20,13 +20,4 @@ public interface Subject {
      * Get the application-wide uniquely identifying principal
      * */
     Principal getPrincipal();
-
-    /**
-     * Authenticate via an auth token
-     * throws UnsupportedAuthenticationMethod
-     * throws InvalidAuthenticationToken
-     * throws SubjectNotFound
-     * throws SubjectDisabled
-     */
-    void authenticate(final AuthToken token);
 }
diff --git a/server/src/main/java/org/opensearch/identity/noop/NoopSubject.java b/server/src/main/java/org/opensearch/identity/noop/NoopSubject.java
@@ -46,12 +46,4 @@ public int hashCode() {
     public String toString() {
         return "NoopSubject(principal=" + getPrincipal() + ")";
     }
-
-    /**
-     * Logs the user in
-     */
-    @Override
-    public void authenticate(AuthToken AuthToken) {
-        // Do nothing as noop subject is always logged in
-    }
 }