Add comments about getCookie method

Signed-off-by: merlinz01 <[email protected]>
opensearch-project · Sep 11, 2024 · 6e08721 · 6e08721
1 parent 87e9d9e
commit 6e08721
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/server/auth/types/jwt/jwt_auth.ts b/server/auth/types/jwt/jwt_auth.ts
@@ -149,6 +149,7 @@ export class JwtAuthentication extends AuthenticationType {
     request: OpenSearchDashboardsRequest<unknown, unknown, unknown, any>,
     authInfo: any
   ): SecuritySessionCookie {
+    // TODO: This logic is only applicable for JWT auth type
     setExtraAuthStorage(
       request,
       this.getBearerToken(request) || '',

diff --git a/server/auth/types/multiple/multi_auth.ts b/server/auth/types/multiple/multi_auth.ts
@@ -163,6 +163,7 @@ export class MultipleAuthentication extends AuthenticationType {
   }
 
   getCookie(request: OpenSearchDashboardsRequest, authInfo: any): SecuritySessionCookie {
+    // TODO: This logic is only applicable for JWT auth type
     for (const handler of this.authHandlers.values()) {
       if (handler.requestIncludesAuthInfo(request)) {
         return handler.getCookie(request, authInfo);