Fixing bug in issuer cache when updating idm type of a tenant. (#23)

issuercache.go

@@ -208,6 +208,9 @@ func (i *MultiIssuerCache) syncCache(newIcs []*IssuerConfig) error {
 			continue
 		}
 
+		// update the annotations always
+		v.issuerConfig.Annotations = newTenantConfig.Annotations
+
 		// found existing cached tenant, check for update
 		newCidIssKey := cacheKey(newTenantConfig.Issuer, newTenantConfig.ClientID)
 		if cidIssKey != newCidIssKey {
@@ -216,7 +219,7 @@ func (i *MultiIssuerCache) syncCache(newIcs []*IssuerConfig) error {
 			delete(i.cache, cidIssKey)
 			// add new entry
 			i.cache[newCidIssKey] = &Issuer{issuerConfig: newTenantConfig}
-			i.log.Info("syncCache - updated tenant in cache", "tenant", tenant, "key", cidIssKey)
+			i.log.Info("syncCache - updated tenant in cache", "tenant", tenant, "key", cidIssKey, "annotations", newTenantConfig.Annotations)
 		}
 
 		// delete entry from newTenantIDMap, as it is already processed
@@ -227,7 +230,7 @@ func (i *MultiIssuerCache) syncCache(newIcs []*IssuerConfig) error {
 	for _, ic := range newTenantIDMap {
 		key := cacheKey(ic.Issuer, ic.ClientID)
 		i.cache[key] = &Issuer{issuerConfig: ic}
-		i.log.Info("syncCache - add tenant to cache", "tenant", ic.Tenant, "key", key)
+		i.log.Info("syncCache - add tenant to cache", "tenant", ic.Tenant, "key", key, "annotations", ic.Annotations)
 	}
 	return nil
 }