Fixes

Signed-off-by: Nils Bandener <nils.bandener@eliatra.com>
opensearch-project · Jun 20, 2024 · d128425 · d128425
1 parent dbf7cd6
commit d128425
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 19 deletions.
diff --git a/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluator.java b/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluator.java
@@ -199,13 +199,15 @@ public PrivilegesEvaluator(
                 SecurityDynamicConfiguration<?> rolesConfiguration = configurationRepository.getConfiguration(CType.ROLES);
 
                 if (rolesConfiguration != null) {
-                    FlattenedActionGroups flattenedActionGroups = actionGroupsConfiguration != null
-                        ? new FlattenedActionGroups(
-                            (SecurityDynamicConfiguration<ActionGroupsV7>) DynamicConfigFactory.addStatics(
-                                actionGroupsConfiguration.deepClone()
-                            )
+                    @SuppressWarnings("unchecked")
+                    SecurityDynamicConfiguration<ActionGroupsV7> actionGroupsWithStatics = actionGroupsConfiguration != null
+                        ? (SecurityDynamicConfiguration<ActionGroupsV7>) DynamicConfigFactory.addStatics(
+                            actionGroupsConfiguration.deepClone()
                         )
-                        : FlattenedActionGroups.EMPTY;
+                        : (SecurityDynamicConfiguration<ActionGroupsV7>) DynamicConfigFactory.addStatics(
+                            SecurityDynamicConfiguration.empty()
+                        );
+                    FlattenedActionGroups flattenedActionGroups = new FlattenedActionGroups(actionGroupsWithStatics);
                     ActionPrivileges actionPrivileges = new ActionPrivileges(
                         DynamicConfigFactory.addStatics(rolesConfiguration.deepClone()),
                         flattenedActionGroups,

diff --git a/src/main/java/org/opensearch/security/securityconf/impl/SecurityDynamicConfiguration.java b/src/main/java/org/opensearch/security/securityconf/impl/SecurityDynamicConfiguration.java
@@ -172,19 +172,6 @@ public static <T> SecurityDynamicConfiguration<T> fromYaml(String yaml, CType ct
         return result;
     }
 
-    /**
-     * For testing only
-     */
-    public static <T> SecurityDynamicConfiguration<T> fromMap(Map<String, Object> map, CType ctype) throws JsonProcessingException {
-        Class<?> implementationClass = ctype.getImplementationClass().get(2);
-        SecurityDynamicConfiguration<T> result = DefaultObjectMapper.objectMapper.convertValue(
-            map,
-            DefaultObjectMapper.getTypeFactory().constructParametricType(SecurityDynamicConfiguration.class, implementationClass)
-        );
-        result.ctype = ctype;
-        return result;
-    }
-
     // for Jackson
     private SecurityDynamicConfiguration() {
         super();