Extend the config cache update logging

Author: pablothedude

src/Surfnet/AzureMfa/Infrastructure/Factory/IdentityProviderFactory.php

@@ -56,14 +56,16 @@ public function __construct(ConfigurationFactory $configurationFactory, Metadata
     public function build(
         InstitutionName $institutionName,
     ): IdentityProviderInterface {
+        $this->logger->info(sprintf('Start updating the identity configuration for: %s', $institutionName->getInstitutionName()));
+
         $entity = $this->configurationFactory->getEntity($institutionName);
 
         if (!$entity instanceof InstitutionConfigurationData) {
             throw new InstitutionNotFoundException('The institution with name "' . $institutionName->getInstitutionName() . '" was not found in the configuration.');
         }
 
         if ($entity->hasMetadataUrl()) {
-            $this->logger->info(sprintf('Fetching metadata for institution: %s', $institutionName->getInstitutionName()));
+            $this->logger->info(sprintf('Update configuration for institution with the metadata url: %s', $institutionName->getInstitutionName()));
 
             try {
                 $identityProvider = $this->metadataIdentityProviderService->fetch($entity);
@@ -72,11 +74,13 @@ public function build(
                 throw $e;
             }
 
-            $this->logger->info(sprintf('Successfully fetched metadata for institution: %s', $institutionName->getInstitutionName()));
+            $this->logger->info(sprintf('Successfully updated with metadata for institution: %s, %d certificates found', $institutionName->getInstitutionName(), count($identityProvider->getCertificates()->getCertificates())));
 
             return $identityProvider;
         }
 
+        $this->logger->info(sprintf('Update configuration for institution with the application config: %s', $institutionName->getInstitutionName()));
+
         if (!$entity->hasCertificates()) {
             throw new InvalidCertificateException('The entity provider must have at least one certificate.');
         }
@@ -86,6 +90,10 @@ public function build(
         $certificates = CertificateCollection::fromStringArray($entity->getCertificates());
         $isAzureAD = $entity->isAzureAd();
 
-        return new AzureMfaIdentityProvider($entityId, $ssoLocation, $certificates, $isAzureAD);
+        $identityProvider = new AzureMfaIdentityProvider($entityId, $ssoLocation, $certificates, $isAzureAD);
+
+        $this->logger->info(sprintf('Successfully updated with config for institution: %s, %d certificates found', $institutionName->getInstitutionName(), count($identityProvider->getCertificates()->getCertificates())));
+
+        return $identityProvider;
     }
 }