keycloak · ahus1 · Jun 17, 2025 · Jun 16, 2025
@@ -138,14 +138,14 @@ protected void initProviderFactories(boolean updateComponentFactory, Map<Class<?
         Stack<ProviderFactory> recursionPrevention = new Stack<>();
 
         for(Map.Entry<Class<? extends Provider>, Map<String, ProviderFactory>>  f : factories.entrySet()) {
-            if (initializedProviders.contains(f.getKey())) {
-                continue;
-            }
             initializeProviders(f.getKey(), factories, initializedProviders, recursionPrevention);
         }
     }
 
-    private void initializeProviders(Class<? extends Provider> provider, Map<Class<? extends Provider>, Map<String, ProviderFactory>> factories, Set<Class<? extends Provider>> intializedProviders, Stack<ProviderFactory> recursionPrevention) {
+    private void initializeProviders(Class<? extends Provider> provider, Map<Class<? extends Provider>, Map<String, ProviderFactory>> factories, Set<Class<? extends Provider>> initializedProviders, Stack<ProviderFactory> recursionPrevention) {
+        if (initializedProviders.contains(provider)) {
+            return;
+        }
         for (ProviderFactory<?> factory : factories.get(provider).values()) {
             if (factory == componentFactoryPF)
                 continue;
@@ -161,11 +161,11 @@ private void initializeProviders(Class<? extends Provider> provider, Map<Class<?
                     throw new RuntimeException("No provider factories exists for provider " + providerDep.getSimpleName() + " required by " + factory.getClass().getName() + " (" + factory.getId() + ")");
                 }
                 recursionPrevention.push(factory);
-                initializeProviders(providerDep, factories, intializedProviders, recursionPrevention);
+                initializeProviders(providerDep, factories, initializedProviders, recursionPrevention);
                 recursionPrevention.pop();
             }
             factory.postInit(this);
-            intializedProviders.add(provider);
+            initializedProviders.add(provider);
         }
     }
 

@@ -10,9 +10,13 @@
 import org.keycloak.provider.Provider;
 import org.keycloak.provider.ProviderFactory;
 import org.keycloak.provider.Spi;
+import org.keycloak.vault.VaultProvider;
+
+import static org.junit.Assert.assertFalse;
 
 import java.util.HashMap;
 import java.util.Map;
+import java.util.Set;
 
 public class DefaultKeycloakSessionFactoryTest {
 
@@ -30,6 +34,41 @@ public void after() {
         Config.init(new Config.SystemPropertiesConfigProvider());
     }
 
+    @Test
+    public void testProviderInitialization() {
+        DefaultKeycloakSessionFactory factory = new DefaultKeycloakSessionFactory() {
+
+            @Override
+            public KeycloakSession create() {
+                return null;
+            }
+        };
+
+        Map<String, ProviderFactory> dependants = Map.of("two", new DummyProviderFactory("two", 2) {
+            @Override
+            public Set<Class<? extends Provider>> dependsOn() {
+                return Set.of(VaultProvider.class);
+            }
+        }, "one", new DummyProviderFactory("one", 0) {
+            @Override
+            public Set<Class<? extends Provider>> dependsOn() {
+                return Set.of(VaultProvider.class);
+            }
+        });
+
+        Map<String, ProviderFactory> vault = Map.of("three", new DummyVaultProviderFactory("three", 3) {
+            boolean init;
+
+            @Override
+            public void postInit(KeycloakSessionFactory factory) {
+                assertFalse(init);
+                init = true;
+            }
+        });
+
+        factory.initProviderFactories(false, Map.of(Provider.class, dependants, VaultProvider.class, vault));
+    }
+
     @Test
     public void defaultProviderFromConfigTest() {
         Map<String, ProviderFactory> map = new HashMap<>(Map.of(
@@ -90,18 +129,34 @@ public Config.Scope scope(String... scope) {
         }
     }
 
-    private class DummyProviderFactory implements ProviderFactory {
-
-        private String id;
-        private int order;
+    private class DummyProviderFactory extends SimpleProviderFactory<Provider> {
 
         public DummyProviderFactory(String id, int order) {
+            super(id, order);
+        }
+
+    }
+
+    private class DummyVaultProviderFactory extends SimpleProviderFactory<VaultProvider> {
+
+        public DummyVaultProviderFactory(String id, int order) {
+            super(id, order);
+        }
+
+    }
+
+    private class SimpleProviderFactory<T extends Provider> implements ProviderFactory<T> {
+
+        String id;
+        int order;
+
+        public SimpleProviderFactory(String id, int order) {
             this.id = id;
             this.order = order;
         }
 
         @Override
-        public Provider create(KeycloakSession session) {
+        public T create(KeycloakSession session) {
             return null;
         }