disallow custom client option when loading from afd

Author: zhiyuanliang-ms

src/common/errorMessages.ts

@@ -20,8 +20,9 @@ export const enum ErrorMessages {
     INVALID_LABEL_FILTER = "The characters '*' and ',' are not supported in label filters.",
     INVALID_TAG_FILTER = "Tag filter must follow the format 'tagName=tagValue'",
     CONNECTION_STRING_OR_ENDPOINT_MISSED = "A connection string or an endpoint with credential must be specified to create a client.",
-    REPLICA_DISCOVERY_NOT_SUPPORTED = "Replica discovery is not supported when loading from Azure Front Door.",
-    LOAD_BALANCING_NOT_SUPPORTED = "Load balancing is not supported when loading from Azure Front Door."
+    AFD_REPLICA_DISCOVERY_NOT_SUPPORTED = "Replica discovery is not supported when loading from Azure Front Door.",
+    AFD_LOAD_BALANCING_NOT_SUPPORTED = "Load balancing is not supported when loading from Azure Front Door.",
+    AFD_CLIENT_OPTIONS_NOT_SUPPORTED = "Custom client options are not supported when loading from Azure Front Door."
 }
 
 export const enum KeyVaultReferenceErrorMessages {

src/load.ts

@@ -78,17 +78,18 @@ export async function loadFromAzureFrontDoor(
     appConfigOptions: AzureAppConfigurationOptions = {}
 ): Promise<AzureAppConfiguration> {
     if (appConfigOptions.replicaDiscoveryEnabled) {
-        throw new ArgumentError(ErrorMessages.REPLICA_DISCOVERY_NOT_SUPPORTED);
+        throw new ArgumentError(ErrorMessages.AFD_REPLICA_DISCOVERY_NOT_SUPPORTED);
     }
     if (appConfigOptions.loadBalancingEnabled) {
-        throw new ArgumentError(ErrorMessages.LOAD_BALANCING_NOT_SUPPORTED);
+        throw new ArgumentError(ErrorMessages.AFD_LOAD_BALANCING_NOT_SUPPORTED);
+    }
+    if (appConfigOptions.clientOptions) {
+        throw new ArgumentError(ErrorMessages.AFD_CLIENT_OPTIONS_NOT_SUPPORTED);
     }
     appConfigOptions.replicaDiscoveryEnabled = false; // Disable replica discovery when loading from Azure Front Door
 
     appConfigOptions.clientOptions = {
-        ...appConfigOptions.clientOptions,
         additionalPolicies: [
-            ...(appConfigOptions.clientOptions?.additionalPolicies || []),
             { policy: new AnonymousRequestPipelinePolicy(), position: "perRetry" },
             { policy: new RemoveSyncTokenPipelinePolicy(), position: "perRetry" }
         ]