Add version validation to prevent unauthorized SDK usage

Added validation checks in UserAgentPolicy and SDK-specific user agent modules to ensure VERSION is never empty, None, or whitespace-only, which would indicate improperly configured or unauthorized SDK usage.

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

Author: Copilot

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

@@ -6,4 +6,8 @@
 
 from ._version import VERSION
 
+# Validate that VERSION is not empty or None to prevent unauthorized SDK usage
+if not VERSION or not isinstance(VERSION, str) or not VERSION.strip():
+    raise ValueError("Invalid SDK version: version must be a non-empty string")
+
 USER_AGENT = "python-appconfiguration-provider/{}".format(VERSION)

sdk/containerregistry/azure-containerregistry/azure/containerregistry/_user_agent.py

@@ -6,4 +6,8 @@
 
 from ._version import VERSION
 
+# Validate that VERSION is not empty or None to prevent unauthorized SDK usage
+if not VERSION or not isinstance(VERSION, str) or not VERSION.strip():
+    raise ValueError("Invalid SDK version: version must be a non-empty string")
+
 USER_AGENT = f"azure-containerregistry/{VERSION}"

sdk/core/azure-core/azure/core/pipeline/policies/_universal.py

@@ -207,6 +207,11 @@ def __init__(self, base_user_agent: Optional[str] = None, **kwargs: Any) -> None
         self.overwrite: bool = kwargs.pop("user_agent_overwrite", False)
         self.use_env: bool = kwargs.pop("user_agent_use_env", True)
         application_id: Optional[str] = kwargs.pop("user_agent", None)
+        
+        # Validate that azcore_version is not empty or None to prevent unauthorized SDK usage
+        if not azcore_version or not isinstance(azcore_version, str) or not azcore_version.strip():
+            raise ValueError("Invalid SDK version: version must be a non-empty string")
+        
         sdk_moniker: str = kwargs.pop("sdk_moniker", "core/{}".format(azcore_version))
 
         if base_user_agent:

sdk/evaluation/azure-ai-evaluation/azure/ai/evaluation/_user_agent.py

@@ -6,6 +6,10 @@
 
 from azure.ai.evaluation._version import VERSION
 
+# Validate that VERSION is not empty or None to prevent unauthorized SDK usage
+if not VERSION or not isinstance(VERSION, str) or not VERSION.strip():
+    raise ValueError("Invalid SDK version: version must be a non-empty string")
+
 
 class UserAgentSingleton:
     __BASE_USER_AGENT: str = "{}/{}".format("azure-ai-evaluation", VERSION)

sdk/formrecognizer/azure-ai-formrecognizer/azure/ai/formrecognizer/_user_agent.py

@@ -5,4 +5,8 @@
 
 from ._version import VERSION
 
+# Validate that VERSION is not empty or None to prevent unauthorized SDK usage
+if not VERSION or not isinstance(VERSION, str) or not VERSION.strip():
+    raise ValueError("Invalid SDK version: version must be a non-empty string")
+
 USER_AGENT = f"ai-formrecognizer/{VERSION}"

sdk/identity/azure-identity/azure/identity/_internal/user_agent.py

@@ -6,4 +6,8 @@
 
 from .._version import VERSION
 
+# Validate that VERSION is not empty or None to prevent unauthorized SDK usage
+if not VERSION or not isinstance(VERSION, str) or not VERSION.strip():
+    raise ValueError("Invalid SDK version: version must be a non-empty string")
+
 USER_AGENT = "azsdk-python-identity/{} Python/{} ({})".format(VERSION, platform.python_version(), platform.platform())

sdk/ml/azure-ai-ml/azure/ai/ml/_user_agent.py

@@ -3,4 +3,8 @@
 # ---------------------------------------------------------
 from azure.ai.ml._version import VERSION
 
+# Validate that VERSION is not empty or None to prevent unauthorized SDK usage
+if not VERSION or not isinstance(VERSION, str) or not VERSION.strip():
+    raise ValueError("Invalid SDK version: version must be a non-empty string")
+
 USER_AGENT = "{}/{}".format("azure-ai-ml", VERSION)

sdk/textanalytics/azure-ai-textanalytics/azure/ai/textanalytics/_user_agent.py

@@ -5,4 +5,8 @@
 
 from ._version import VERSION
 
+# Validate that VERSION is not empty or None to prevent unauthorized SDK usage
+if not VERSION or not isinstance(VERSION, str) or not VERSION.strip():
+    raise ValueError("Invalid SDK version: version must be a non-empty string")
+
 USER_AGENT = f"ai-textanalytics/{VERSION}"