update

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

Hide file tree

Showing 2 changed files with 6 additions and 12 deletions.
diff --git a/src/main/java/org/opensearch/security/OpenSearchSecurityPlugin.java b/src/main/java/org/opensearch/security/OpenSearchSecurityPlugin.java
@@ -1953,18 +1953,6 @@ public TokenManager getTokenManager() {
         return tokenManager;
     }
 
-    /**
-     * Handles tracking the active subject when requests are made using an auth token
-     * ~NOT right~
-     * @param subject
-     * @param authToken
-     */
-    @Override
-    public void authenticate(Subject subject, AuthToken authToken) {
-        this.subject = (SecuritySubject) subject;
-        this.subject.setThreadContext(threadPool.getThreadContext());
-    }
-
     public static class GuiceHolder implements LifecycleComponent {
 
         private static RepositoriesService repositoriesService;

diff --git a/src/main/java/org/opensearch/security/identity/SecuritySubject.java b/src/main/java/org/opensearch/security/identity/SecuritySubject.java
@@ -15,6 +15,7 @@
 import org.opensearch.common.util.concurrent.ThreadContext;
 import org.opensearch.identity.NamedPrincipal;
 import org.opensearch.identity.Subject;
+import org.opensearch.identity.tokens.AuthToken;
 import org.opensearch.security.support.ConfigConstants;
 import org.opensearch.security.user.User;
 
@@ -39,4 +40,9 @@ public Principal getPrincipal() {
 		}
 		return new NamedPrincipal(user.getName());
 	}
+
+	@Override
+	public void authenticate(AuthToken authToken) {
+		// TODO: How is this supposed to work?
+	}
 }