Mark requests to /bundles default authenticated (#512)

opensearch-project · Oct 6, 2020 · 652746e · 652746e
1 parent c4571a6
commit 652746e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/public/apps/configuration/utils/role-list-utils.tsx b/public/apps/configuration/utils/role-list-utils.tsx
@@ -73,7 +73,7 @@ export function transformRoleData(rawRoleData: any, rawRoleMappingData: any): Ro
       .flatten()
       .compact()
       .value() as string[],
-    internalUsers: rawRoleMappingData.data[k || ''].users || [],
+    internalUsers: rawRoleMappingData.data[k || '']?.users || [],
     backendRoles: rawRoleMappingData.data[k || '']?.backend_roles || [],
   }));
 }

diff --git a/server/auth/types/authentication_type.ts b/server/auth/types/authentication_type.ts
@@ -77,7 +77,7 @@ export abstract class AuthenticationType implements IAuthenticationType {
   public authHandler: AuthenticationHandler = async (request, response, toolkit) => {
     // allow access to assets
     if (request.url.pathname && request.url.pathname.startsWith('/bundles/')) {
-      return toolkit.notHandled();
+      return toolkit.authenticated();
     }
 
     // skip auth for APIs that do not require auth