App Configuration Provider - Key Vault Refresh (#41882)

* Sync refresh changes

* Key Vault Refresh

* adding tests and fixing sync refresh

* Updating Async

* Fixed Async Tests

* Updated tests and change log

* Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>

* Fixing merge issue

* Updating comments

* Updating secret refresh

* Update _azureappconfigurationproviderasync.py

* Fixing Optional Endpoint

* fix mypy issue

* fixing async test

* mixing merge

* fixing test after merge

* Update testcase.py

* Secret Provider Base

* removing unused imports

* updating exception

* updating resolve key vault references

* Review comments

* fixing tests

* tox updates

* Updating Tests

* Updating Async to be the same as sync

* Fixing formatting

* fixing tox and unneeded ""

* fixing tox items

* fix cspell + tests recording

* Update test_async_secret_provider.py

* Post Merge updates

* Move cache to shared code

* removed unneeded disabled

* Update Secret Provider

* Updating usage

* Update assets.json

* Updated to make secret refresh update dictionary

* removing _secret_version_cache

* Update assets.json

* Update _secret_provider_base.py

---------

Co-authored-by: Copilot <[email protected]>

Author: mrm9084

sdk/appconfiguration/azure-appconfiguration-provider/CHANGELOG.md

@@ -4,10 +4,15 @@
 
 ### Features Added
 
+* Added support for forced refresh of configurations when using Key Vault references. Adds `secret_refresh_interval` to the `AzureAppConfigurationProvider.load` method. This allows the provider to refresh Key Vault secrets at a specified interval. Is set to 60 seconds by default, and can only be set if using Key Vault references.
+* Added support for async `on_refresh_success`.
+
 ### Breaking Changes
 
 ### Bugs Fixed
 
+* Fixed a bug where feature flags were using the configuration refresh timer instead of the feature flag refresh timer.
+
 ### Other Changes
 
 ## 2.2.0 (2025-08-08)

sdk/appconfiguration/azure-appconfiguration-provider/assets.json

@@ -2,5 +2,5 @@
   "AssetsRepo": "Azure/azure-sdk-assets",
   "AssetsRepoPrefixPath": "python",
   "TagPrefix": "python/appconfiguration/azure-appconfiguration-provider",
-  "Tag": "python/appconfiguration/azure-appconfiguration-provider_c68d337f0e"
+  "Tag": "python/appconfiguration/azure-appconfiguration-provider_b13d43b82a"
 }

sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/_azureappconfigurationprovider.py

@@ -22,17 +22,17 @@
     SecretReferenceConfigurationSetting,
 )
 from azure.core.exceptions import AzureError, HttpResponseError
-from azure.keyvault.secrets import SecretClient, KeyVaultSecretIdentifier
 from ._models import AzureAppConfigurationKeyVaultOptions, SettingSelector
+from ._key_vault._secret_provider import SecretProvider
 from ._constants import (
     FEATURE_MANAGEMENT_KEY,
     FEATURE_FLAG_KEY,
 )
 from ._azureappconfigurationproviderbase import (
     AzureAppConfigurationProviderBase,
+    update_correlation_context_header,
     delay_failure,
     sdk_allowed_kwargs,
-    update_correlation_context_header,
 )
 from ._client_manager import ConfigurationClientManager, _ConfigurationClientWrapper as ConfigurationClient
 from ._user_agent import USER_AGENT
@@ -246,46 +246,6 @@ def _buildprovider(
     return AzureAppConfigurationProvider(**kwargs)
 
 
-def _resolve_keyvault_reference(
-    config: "SecretReferenceConfigurationSetting", provider: "AzureAppConfigurationProvider"
-) -> str:
-    # pylint:disable=protected-access
-    if not (provider._keyvault_credential or provider._keyvault_client_configs or provider._secret_resolver):
-        raise ValueError(
-            """
-            Either a credential to Key Vault, custom Key Vault client, or a secret resolver must be set to resolve Key
-             Vault references.
-            """
-        )
-
-    if config.secret_id is None:
-        raise ValueError("Key Vault reference must have a uri value.")
-
-    keyvault_identifier = KeyVaultSecretIdentifier(config.secret_id)
-
-    vault_url = keyvault_identifier.vault_url + "/"
-
-    # pylint:disable=protected-access
-    referenced_client = provider._secret_clients.get(vault_url, None)
-
-    vault_config = provider._keyvault_client_configs.get(vault_url, {})
-    credential = vault_config.pop("credential", provider._keyvault_credential)
-
-    if referenced_client is None and credential is not None:
-        referenced_client = SecretClient(vault_url=vault_url, credential=credential, **vault_config)
-        provider._secret_clients[vault_url] = referenced_client
-
-    if referenced_client:
-        secret_value = referenced_client.get_secret(keyvault_identifier.name, version=keyvault_identifier.version).value
-        if secret_value is not None:
-            return secret_value
-
-    if provider._secret_resolver:
-        return provider._secret_resolver(config.secret_id)
-
-    raise ValueError("No Secret Client found for Key Vault reference %s" % (vault_url))
-
-
 class AzureAppConfigurationProvider(AzureAppConfigurationProviderBase):  # pylint: disable=too-many-instance-attributes
     """
     Provides a dictionary-like interface to Azure App Configuration settings. Enables loading of sets of configuration
@@ -309,8 +269,8 @@ def __init__(self, **kwargs: Any) -> None:
         max_backoff: int = min(kwargs.pop("max_backoff", 600), interval)
 
         self._replica_client_manager = ConfigurationClientManager(
-            connection_string=kwargs.pop("connection_string", None),
-            endpoint=kwargs.pop("endpoint", None),
+            connection_string=kwargs.pop("connection_string"),
+            endpoint=kwargs.pop("endpoint"),
             credential=kwargs.pop("credential", None),
             user_agent=user_agent,
             retry_total=kwargs.pop("retry_total", 2),
@@ -321,7 +281,7 @@ def __init__(self, **kwargs: Any) -> None:
             load_balancing_enabled=kwargs.pop("load_balancing_enabled", False),
             **kwargs,
         )
-        self._secret_clients: Dict[str, SecretClient] = {}
+        self._secret_provider = SecretProvider(**kwargs)
         self._on_refresh_success: Optional[Callable] = kwargs.pop("on_refresh_success", None)
         self._on_refresh_error: Optional[Callable[[Exception], None]] = kwargs.pop("on_refresh_error", None)
         self._configuration_mapper: Optional[Callable] = kwargs.pop("configuration_mapper", None)
@@ -334,7 +294,7 @@ def _attempt_refresh(self, client: ConfigurationClient, replica_count: int, is_f
             replica_count,
             self._feature_flag_enabled,
             self._feature_filter_usage,
-            self._uses_key_vault,
+            self._secret_provider.uses_key_vault,
             self._uses_load_balancing,
             is_failover_request,
             self._uses_ai_configuration,
@@ -424,6 +384,11 @@ def refresh(self, **kwargs) -> None:
         exception: Optional[Exception] = None
         is_failover_request = False
         try:
+            if (
+                self._secret_provider.secret_refresh_timer
+                and self._secret_provider.secret_refresh_timer.needs_refresh()
+            ):
+                self._dict.update(self._secret_provider.refresh_secrets())
             self._replica_client_manager.refresh_clients()
             self._replica_client_manager.find_active_clients()
             replica_count = self._replica_client_manager.get_client_count() - 1
@@ -465,7 +430,7 @@ def _load_all(self, **kwargs):
                 replica_count,
                 self._feature_flag_enabled,
                 self._feature_filter_usage,
-                self._uses_key_vault,
+                self._secret_provider.uses_key_vault,
                 self._uses_load_balancing,
                 is_failover_request,
                 self._uses_ai_configuration,
@@ -520,9 +485,13 @@ def _load_all(self, **kwargs):
         raise exception
 
     def _process_configurations(self, configuration_settings: List[ConfigurationSetting]) -> Dict[str, Any]:
+        # configuration_settings can contain duplicate keys, but they are in priority order, i.e. later settings take
+        # precedence. Only process the settings with the highest priority (i.e. the last one in the list).
+        unique_settings = self._deduplicate_settings(configuration_settings)
+
         configuration_settings_processed = {}
         feature_flags_processed = []
-        for settings in configuration_settings:
+        for settings in unique_settings.values():
             if self._configuration_mapper:
                 # If a map function is provided, use it to process the configuration setting
                 self._configuration_mapper(settings)
@@ -541,7 +510,7 @@ def _process_configurations(self, configuration_settings: List[ConfigurationSett
 
     def _process_key_value(self, config: ConfigurationSetting) -> Any:
         if isinstance(config, SecretReferenceConfigurationSetting):
-            return _resolve_keyvault_reference(config, self)
+            return self._secret_provider.resolve_keyvault_reference(config)
         # Use the base class helper method for non-KeyVault processing
         return self._process_key_value_base(config)
 
@@ -558,17 +527,14 @@ def close(self) -> None:
         """
         Closes the connection to Azure App Configuration.
         """
-        for client in self._secret_clients.values():
-            client.close()
+        self._secret_provider.close()
         self._replica_client_manager.close()
 
     def __enter__(self) -> "AzureAppConfigurationProvider":
         self._replica_client_manager.__enter__()
-        for client in self._secret_clients.values():
-            client.__enter__()
+        self._secret_provider.__enter__()
         return self
 
     def __exit__(self, *args) -> None:
         self._replica_client_manager.__exit__()
-        for client in self._secret_clients.values():
-            client.__exit__()
+        self._secret_provider.__exit__()

sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/_azureappconfigurationproviderbase.py

@@ -290,20 +290,11 @@ def __init__(self, **kwargs: Any) -> None:
 
         trim_prefixes: List[str] = kwargs.pop("trim_prefixes", [])
         self._trim_prefixes: List[str] = sorted(trim_prefixes, key=len, reverse=True)
-
         refresh_on: List[Tuple[str, str]] = kwargs.pop("refresh_on", None) or []
         self._watched_settings: Dict[Tuple[str, str], Optional[str]] = {
             _build_watched_setting(s): None for s in refresh_on
         }
         self._refresh_timer: _RefreshTimer = _RefreshTimer(**kwargs)
-        self._keyvault_credential = kwargs.pop("keyvault_credential", None)
-        self._secret_resolver = kwargs.pop("secret_resolver", None)
-        self._keyvault_client_configs = kwargs.pop("keyvault_client_configs", {})
-        self._uses_key_vault = (
-            self._keyvault_credential is not None
-            or (self._keyvault_client_configs is not None and len(self._keyvault_client_configs) > 0)
-            or self._secret_resolver is not None
-        )
         self._feature_flag_enabled = kwargs.pop("feature_flag_enabled", False)
         self._feature_flag_selectors = kwargs.pop("feature_flag_selectors", [SettingSelector(key_filter="*")])
         self._watched_feature_flags: Dict[Tuple[str, str], Optional[str]] = {}
@@ -598,3 +589,18 @@ def _update_watched_feature_flags(
         for feature_flag in feature_flags:
             watched_feature_flags[(feature_flag.key, feature_flag.label)] = feature_flag.etag
         return watched_feature_flags
+
+    def _deduplicate_settings(
+        self, configuration_settings: List[ConfigurationSetting]
+    ) -> Dict[str, ConfigurationSetting]:
+        """
+        Deduplicates configuration settings by key.
+
+        :param List[ConfigurationSetting] configuration_settings: The list of configuration settings to deduplicate
+        :return: A dictionary mapping keys to their unique configuration settings
+        :rtype: Dict[str, ConfigurationSetting]
+        """
+        unique_settings: Dict[str, ConfigurationSetting] = {}
+        for settings in configuration_settings:
+            unique_settings[settings.key] = settings
+        return unique_settings

sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/_key_vault/__init__.py

@@ -0,0 +1,13 @@
+# ------------------------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All rights reserved.
+# Licensed under the MIT License. See License.txt in the project root for
+# license information.
+# -------------------------------------------------------------------------
+
+from ._secret_provider import SecretProvider
+from ._secret_provider_base import _SecretProviderBase
+
+__all__ = [
+    "SecretProvider",
+    "_SecretProviderBase",
+]

sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/_key_vault/_secret_provider.py

@@ -0,0 +1,87 @@
+# ------------------------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All rights reserved.
+# Licensed under the MIT License. See License.txt in the project root for
+# license information.
+# -------------------------------------------------------------------------
+from typing import Mapping, Any, Dict
+from azure.appconfiguration import SecretReferenceConfigurationSetting  # type:ignore # pylint:disable=no-name-in-module
+from azure.keyvault.secrets import SecretClient, KeyVaultSecretIdentifier
+from azure.core.exceptions import ServiceRequestError
+from ._secret_provider_base import _SecretProviderBase
+
+JSON = Mapping[str, Any]
+
+
+class SecretProvider(_SecretProviderBase):
+
+    def __init__(self, **kwargs: Any) -> None:
+        super().__init__(**kwargs)
+        self._secret_clients: Dict[str, SecretClient] = {}
+        self._keyvault_credential = kwargs.pop("keyvault_credential", None)
+        self._secret_resolver = kwargs.pop("secret_resolver", None)
+        self._keyvault_client_configs = kwargs.pop("keyvault_client_configs", {})
+
+    def resolve_keyvault_reference(self, config: SecretReferenceConfigurationSetting) -> str:
+        keyvault_identifier, vault_url = self.resolve_keyvault_reference_base(config)
+        if keyvault_identifier.source_id in self._secret_cache:
+            _, _, value = self._secret_cache[keyvault_identifier.source_id]
+            return value
+
+        return self.__get_secret_value(config.key, keyvault_identifier, vault_url)
+
+    def refresh_secrets(self) -> Dict[str, Any]:
+        secrets = {}
+        if self.secret_refresh_timer:
+            original_cache = self._secret_cache.copy()
+            self._secret_cache.clear()
+            for source_id, (secret_identifier, key, _) in original_cache.items():
+                value = self.__get_secret_value(key, secret_identifier, secret_identifier.vault_url + "/")
+                self._secret_cache[source_id] = (
+                    secret_identifier,
+                    key,
+                    value,
+                )
+                secrets[key] = value
+            self.secret_refresh_timer.reset()
+        return secrets
+
+    def __get_secret_value(self, key: str, secret_identifier: KeyVaultSecretIdentifier, vault_url: str) -> str:
+        referenced_client = self._secret_clients.get(vault_url, None)
+
+        vault_config = self._keyvault_client_configs.get(vault_url, {})
+        credential = vault_config.pop("credential", self._keyvault_credential)
+
+        if referenced_client is None and credential is not None:
+            referenced_client = SecretClient(vault_url=vault_url, credential=credential, **vault_config)
+            self._secret_clients[vault_url] = referenced_client
+
+        secret_value = None
+
+        if referenced_client:
+            try:
+                secret_value = referenced_client.get_secret(
+                    secret_identifier.name, version=secret_identifier.version
+                ).value
+            except ServiceRequestError as e:
+                raise ValueError("Failed to retrieve secret from Key Vault") from e
+
+        if self._secret_resolver and secret_value is None:
+            secret_value = self._secret_resolver(secret_identifier.source_id)
+
+        return self._cache_value(key, secret_identifier, secret_value)
+
+    def close(self) -> None:
+        """
+        Closes the connection to Azure App Configuration.
+        """
+        for client in self._secret_clients.values():
+            client.close()
+
+    def __enter__(self) -> "SecretProvider":
+        for client in self._secret_clients.values():
+            client.__enter__()
+        return self
+
+    def __exit__(self, *args) -> None:
+        for client in self._secret_clients.values():
+            client.__exit__()

sdk/appconfiguration/azure-appconfiguration-provider/azure/appconfiguration/provider/_key_vault/_secret_provider_base.py

@@ -0,0 +1,67 @@
+# ------------------------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All rights reserved.
+# Licensed under the MIT License. See License.txt in the project root for
+# license information.
+# -------------------------------------------------------------------------
+from typing import (
+    Mapping,
+    Any,
+    TypeVar,
+    Optional,
+    Dict,
+    Tuple,
+)
+from azure.appconfiguration import SecretReferenceConfigurationSetting  # type:ignore # pylint:disable=no-name-in-module
+from azure.keyvault.secrets import KeyVaultSecretIdentifier
+from .._azureappconfigurationproviderbase import _RefreshTimer
+
+JSON = Mapping[str, Any]
+_T = TypeVar("_T")
+
+
+class _SecretProviderBase:
+
+    def __init__(self, **kwargs: Any) -> None:
+        # [source_id, (KeyVaultSecretIdentifier, key, value)]
+        self._secret_cache: Dict[str, Tuple[KeyVaultSecretIdentifier, str, str]] = {}
+        self.uses_key_vault = (
+            "keyvault_credential" in kwargs
+            or ("keyvault_client_configs" in kwargs and len(kwargs.get("keyvault_client_configs", {})) > 0)
+            or "secret_resolver" in kwargs
+        )
+
+        if kwargs.get("secret_refresh_interval", 60) < 1:
+            raise ValueError("Secret refresh interval must be greater than 1 second.")
+
+        self.secret_refresh_timer: Optional[_RefreshTimer] = (
+            _RefreshTimer(refresh_interval=kwargs.pop("secret_refresh_interval", 60))
+            if self.uses_key_vault and "secret_refresh_interval" in kwargs
+            else None
+        )
+
+    def _cache_value(self, key: str, keyvault_identifier: KeyVaultSecretIdentifier, secret_value: Any) -> str:
+        if secret_value:
+            self._secret_cache[keyvault_identifier.source_id] = (keyvault_identifier, key, secret_value)
+            return secret_value
+
+        raise ValueError("No Secret Client found for Key Vault reference %s" % (keyvault_identifier.vault_url))
+
+    def resolve_keyvault_reference_base(
+        self, config: SecretReferenceConfigurationSetting
+    ) -> Tuple[KeyVaultSecretIdentifier, str]:
+        if not self.uses_key_vault:
+            raise ValueError(
+                """
+                Either a credential to Key Vault, custom Key Vault client, or a secret resolver must be set to resolve
+                Key Vault references.
+                """
+            )
+
+        if config.secret_id is None:
+            raise ValueError("Key Vault reference must have a uri value.")
+
+        keyvault_identifier = KeyVaultSecretIdentifier(config.secret_id)
+
+        vault_url = keyvault_identifier.vault_url + "/"
+
+        return keyvault_identifier, vault_url