Update IdentityProviderManager.java

wso2 · Oct 16, 2024 · 2656cc2 · 2656cc2
1 parent 5911bff
commit 2656cc2
Showing 1 changed file with 4 additions and 8 deletions.
diff --git a/...rg.wso2.carbon.idp.mgt/src/main/java/org/wso2/carbon/idp/mgt/IdentityProviderManager.java b/...rg.wso2.carbon.idp.mgt/src/main/java/org/wso2/carbon/idp/mgt/IdentityProviderManager.java
@@ -2672,15 +2672,11 @@ private void resolveAuthenticatorDefinedByProperty(IdentityProvider idp, boolean
         }
 
         /* For existing federated authenticators, disregard any value provided in the request payload.
-         Instead, resolve and retrieve the 'definedBy' type of the corresponding existing authenticator.
-         If the authenticator config is present in the ApplicationAuthenticatorService list, return its type,
-         if not return USER. */
+         Instead, resolve and retrieve the 'definedBy' type of the corresponding existing authenticator. */
         for (FederatedAuthenticatorConfig federatedAuthConfig : idp.getFederatedAuthenticatorConfigs()) {
-            if (federatedAuthConfig.getDefinedByType() == null) {
-                FederatedAuthenticatorConfig authenticatorConfig = ApplicationAuthenticatorService.getInstance()
-                        .getFederatedAuthenticatorByName(federatedAuthConfig.getName());
-                federatedAuthConfig.setDefinedByType(authenticatorConfig.getDefinedByType());
-            }
+            FederatedAuthenticatorConfig authenticatorConfig = ApplicationAuthenticatorService.getInstance()
+                    .getFederatedAuthenticatorByName(federatedAuthConfig.getName());
+            federatedAuthConfig.setDefinedByType(authenticatorConfig.getDefinedByType());
         }
     }
 }