Skip to content

Commit 6d8821c

Browse files
committed
Replace addIdPCacheOnRead with addIdPCache for write paths
1 parent 7dedaa3 commit 6d8821c

File tree

1 file changed

+1
-25
lines changed

1 file changed

+1
-25
lines changed

components/idp-mgt/org.wso2.carbon.idp.mgt/src/main/java/org/wso2/carbon/idp/mgt/dao/CacheBackedIdPMgtDAO.java

Lines changed: 1 addition & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -338,7 +338,7 @@ public IdentityProvider getIdPByResourceId(String resourceId, int tenantId, Stri
338338
tenantId, tenantDomain);
339339

340340
if (identityProvider != null) {
341-
addIdPCacheOnRead(identityProvider, tenantDomain);
341+
addIdPCache(identityProvider, tenantDomain);
342342
} else {
343343
if (log.isDebugEnabled()) {
344344
log.debug(String.format("No IDP found with resource ID: %s either in cache or DB", resourceId));
@@ -804,30 +804,6 @@ public void addIdPCache(IdentityProvider identityProvider, String tenantDomain)
804804
}
805805
}
806806

807-
/**
808-
* Add new cache entries for IDP against cache keys: name, home-realm-id and resource-id.
809-
*
810-
* @param identityProvider Identity Provider information.
811-
* @param tenantDomain Tenant domain of IDP.
812-
*/
813-
public void addIdPCacheOnRead(IdentityProvider identityProvider, String tenantDomain) {
814-
815-
if (identityProvider != null) {
816-
if (log.isDebugEnabled()) {
817-
log.debug("Adding new entry for Identity Provider: '" + identityProvider.getIdentityProviderName() +
818-
"' to cache.");
819-
}
820-
IdPNameCacheKey idPNameCacheKey = new IdPNameCacheKey(identityProvider.getIdentityProviderName());
821-
idPCacheByName.addToCacheOnRead(idPNameCacheKey, new IdPCacheEntry(identityProvider), tenantDomain);
822-
if (identityProvider.getHomeRealmId() != null) {
823-
IdPHomeRealmIdCacheKey idPHomeRealmIdCacheKey = new IdPHomeRealmIdCacheKey(
824-
identityProvider.getHomeRealmId());
825-
idPCacheByHRI.addToCacheOnRead(idPHomeRealmIdCacheKey, new IdPCacheEntry(identityProvider), tenantDomain);
826-
}
827-
IdPResourceIdCacheKey idPResourceIdCacheKey = new IdPResourceIdCacheKey(identityProvider.getResourceId());
828-
idPCacheByResourceId.addToCacheOnRead(idPResourceIdCacheKey, new IdPCacheEntry(identityProvider), tenantDomain);
829-
}
830-
}
831807

832808
public void clearIdpCache(String idPName, int tenantId, String tenantDomain)
833809
throws IdentityProviderManagementException {

0 commit comments

Comments
 (0)