Bugfix

Signed-off-by: Nils Bandener <nils.bandener@eliatra.com>
opensearch-project · Sep 21, 2024 · 58d03c2 · 58d03c2
1 parent 490d3c9
commit 58d03c2
Showing 1 changed file with 19 additions and 7 deletions.
diff --git a/src/main/java/org/opensearch/security/securityconf/impl/CType.java b/src/main/java/org/opensearch/security/securityconf/impl/CType.java
@@ -63,45 +63,50 @@ public class CType<T> implements Comparable<CType<?>> {
     private static Map<Integer, CType<?>> ordToInstanceMap = new HashMap<>();
 
     public static final CType<ActionGroupsV7> ACTIONGROUPS = new CType<>(
+        "actiongroups",
         "action_groups",
         ActionGroupsV7.class,
         0,
         false,
         new OldConfigVersion<>(1, ActionGroupsV6.class, ActionGroupsV7::new)
     );
-    public static final CType<AllowlistingSettings> ALLOWLIST = new CType<>("allowlist", AllowlistingSettings.class, 1, true);
-    public static final CType<AuditConfig> AUDIT = new CType<>("audit", AuditConfig.class, 2, true);
+    public static final CType<AllowlistingSettings> ALLOWLIST = new CType<>("allowlist", "allowlist", AllowlistingSettings.class, 1, true);
+    public static final CType<AuditConfig> AUDIT = new CType<>("audit", "audit", AuditConfig.class, 2, true);
     public static final CType<ConfigV7> CONFIG = new CType<>(
+        "config",
         "config",
         ConfigV7.class,
         3,
         false,
         new OldConfigVersion<>(1, ConfigV6.class, ConfigV7::new)
     );
     public static final CType<InternalUserV7> INTERNALUSERS = new CType<>(
+        "internalusers",
         "internal_users",
         InternalUserV7.class,
         4,
         false,
         new OldConfigVersion<>(1, InternalUserV6.class, InternalUserV7::new)
     );
-    public static final CType<NodesDn> NODESDN = new CType<>("nodes_dn", NodesDn.class, 5, true);
+    public static final CType<NodesDn> NODESDN = new CType<>("nodesdn", "nodes_dn", NodesDn.class, 5, true);
     public static final CType<RoleV7> ROLES = new CType<>(
+        "roles",
         "roles",
         RoleV7.class,
         6,
         false,
         new OldConfigVersion<>(1, RoleV6.class, RoleV7::new)
     );
     public static final CType<RoleMappingsV7> ROLESMAPPING = new CType<>(
+        "rolesmapping",
         "roles_mapping",
         RoleMappingsV7.class,
         7,
         false,
         new OldConfigVersion<>(1, RoleMappingsV6.class, RoleMappingsV7::new)
     );
-    public static final CType<TenantV7> TENANTS = new CType<>("tenants", TenantV7.class, 8, false);
-    public static final CType<WhitelistingSettings> WHITELIST = new CType<>("whitelist", WhitelistingSettings.class, 9, true);
+    public static final CType<TenantV7> TENANTS = new CType<>("tenants", "tenants", TenantV7.class, 8, false);
+    public static final CType<WhitelistingSettings> WHITELIST = new CType<>("whitelist", "whitelist", WhitelistingSettings.class, 9, true);
 
     private final String name;
     private final Class<T> configClass;
@@ -112,11 +117,18 @@ public class CType<T> implements Comparable<CType<?>> {
 
     @SafeVarargs
     @SuppressWarnings("varargs")
-    private CType(String name, Class<T> configClass, int ord, boolean emptyIfMissing, OldConfigVersion<?, T>... oldConfigVersions) {
+    private CType(
+        String name,
+        String configFileName,
+        Class<T> configClass,
+        int ord,
+        boolean emptyIfMissing,
+        OldConfigVersion<?, T>... oldConfigVersions
+    ) {
         this.name = name;
         this.configClass = configClass;
         this.ord = ord;
-        this.configFileName = name + ".yml";
+        this.configFileName = configFileName + ".yml";
         this.emptyIfMissing = emptyIfMissing;
         this.oldConfigVersions = oldConfigVersions;