Fix spelling, pylint

sdk/keyvault/azure-keyvault-administration/azure/keyvault/administration/_internal/async_challenge_auth_policy.py

@@ -17,9 +17,10 @@
 from copy import deepcopy
 import time
 from typing import Any, Awaitable, Callable, Optional, overload, TypeVar, Union
-from typing_extensions import ParamSpec
 from urllib.parse import urlparse
 
+from typing_extensions import ParamSpec
+
 from azure.core.credentials import AccessToken
 from azure.core.credentials_async import AsyncTokenCredential
 from azure.core.pipeline import PipelineRequest, PipelineResponse
@@ -199,7 +200,7 @@ async def on_challenge(self, request: PipelineRequest, response: PipelineRespons
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")

sdk/keyvault/azure-keyvault-administration/azure/keyvault/administration/_internal/challenge_auth_policy.py

@@ -194,7 +194,7 @@ def on_challenge(self, request: PipelineRequest, response: PipelineResponse) ->
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")

sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_shared/async_challenge_auth_policy.py

@@ -17,9 +17,10 @@
 from copy import deepcopy
 import time
 from typing import Any, Awaitable, Callable, Optional, overload, TypeVar, Union
-from typing_extensions import ParamSpec
 from urllib.parse import urlparse
 
+from typing_extensions import ParamSpec
+
 from azure.core.credentials import AccessToken
 from azure.core.credentials_async import AsyncTokenCredential
 from azure.core.pipeline import PipelineRequest, PipelineResponse
@@ -199,7 +200,7 @@ async def on_challenge(self, request: PipelineRequest, response: PipelineRespons
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")

sdk/keyvault/azure-keyvault-certificates/azure/keyvault/certificates/_shared/challenge_auth_policy.py

@@ -194,7 +194,7 @@ def on_challenge(self, request: PipelineRequest, response: PipelineResponse) ->
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")

sdk/keyvault/azure-keyvault-keys/azure/keyvault/keys/_shared/async_challenge_auth_policy.py

@@ -17,9 +17,10 @@
 from copy import deepcopy
 import time
 from typing import Any, Awaitable, Callable, Optional, overload, TypeVar, Union
-from typing_extensions import ParamSpec
 from urllib.parse import urlparse
 
+from typing_extensions import ParamSpec
+
 from azure.core.credentials import AccessToken
 from azure.core.credentials_async import AsyncTokenCredential
 from azure.core.pipeline import PipelineRequest, PipelineResponse
@@ -199,7 +200,7 @@ async def on_challenge(self, request: PipelineRequest, response: PipelineRespons
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")

sdk/keyvault/azure-keyvault-keys/azure/keyvault/keys/_shared/challenge_auth_policy.py

@@ -194,7 +194,7 @@ def on_challenge(self, request: PipelineRequest, response: PipelineResponse) ->
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")

sdk/keyvault/azure-keyvault-keys/tests/test_challenge_auth.py

@@ -615,7 +615,7 @@ def get_token(*_, **kwargs):
     claim = '{"access_token": {"foo": "bar"}}'
     # Claim token is a string of the base64 encoding of the claim
     claim_token = base64.b64encode(claim.encode()).decode()
-    # Note that no resource or scope is necessarily proovided in a CAE challenge
+    # Note that no resource or scope is necessarily provided in a CAE challenge
     challenge = f'Bearer realm="", {url}, {cid}, {err}, claims="{claim_token}"'
 
     claims_challenge = Mock(status_code=401, headers={"WWW-Authenticate": challenge})
@@ -693,7 +693,7 @@ def get_token(*_, **kwargs):
     claim = '{"access_token": {"foo": "bar"}}'
     # Claim token is a string of the base64 encoding of the claim
     claim_token = base64.b64encode(claim.encode()).decode()
-    # Note that no resource or scope is necessarily proovided in a CAE challenge
+    # Note that no resource or scope is necessarily provided in a CAE challenge
     challenge = f'Bearer realm="", {url}, {cid}, {err}, claims="{claim_token}"'
 
     claims_challenge = Mock(status_code=401, headers={"WWW-Authenticate": challenge})

sdk/keyvault/azure-keyvault-keys/tests/test_challenge_auth_async.py

@@ -580,7 +580,7 @@ async def get_token(*_, **kwargs):
     claim = '{"access_token": {"foo": "bar"}}'
     # Claim token is a string of the base64 encoding of the claim
     claim_token = base64.b64encode(claim.encode()).decode()
-    # Note that no resource or scope is necessarily proovided in a CAE challenge
+    # Note that no resource or scope is necessarily provided in a CAE challenge
     challenge = f'Bearer realm="", {url}, {cid}, {err}, claims="{claim_token}"'
 
     # get_token is called for the first KV challenge and CAE challenge, but not the second KV challenge
@@ -660,7 +660,7 @@ async def get_token(*_, **kwargs):
     claim = '{"access_token": {"foo": "bar"}}'
     # Claim token is a string of the base64 encoding of the claim
     claim_token = base64.b64encode(claim.encode()).decode()
-    # Note that no resource or scope is necessarily proovided in a CAE challenge
+    # Note that no resource or scope is necessarily provided in a CAE challenge
     challenge = f'Bearer realm="", {url}, {cid}, {err}, claims="{claim_token}"'
 
     claims_challenge = Mock(status_code=401, headers={"WWW-Authenticate": challenge})

sdk/keyvault/azure-keyvault-secrets/azure/keyvault/secrets/_shared/async_challenge_auth_policy.py

@@ -17,9 +17,10 @@
 from copy import deepcopy
 import time
 from typing import Any, Awaitable, Callable, Optional, overload, TypeVar, Union
-from typing_extensions import ParamSpec
 from urllib.parse import urlparse
 
+from typing_extensions import ParamSpec
+
 from azure.core.credentials import AccessToken
 from azure.core.credentials_async import AsyncTokenCredential
 from azure.core.pipeline import PipelineRequest, PipelineResponse
@@ -199,7 +200,7 @@ async def on_challenge(self, request: PipelineRequest, response: PipelineRespons
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")

sdk/keyvault/azure-keyvault-secrets/azure/keyvault/secrets/_shared/challenge_auth_policy.py

@@ -194,7 +194,7 @@ def on_challenge(self, request: PipelineRequest, response: PipelineResponse) ->
                 # Use the old scope for CAE challenges. The parsing will succeed here since it did before
                 if challenge.claims and old_scope:
                     resource_domain = urlparse(old_scope).netloc
-                    challenge._parameters["scope"] = old_scope
+                    challenge._parameters["scope"] = old_scope  # pylint:disable=protected-access
                     challenge.tenant_id = old_tenant
                 else:
                     raise ValueError(f"The challenge contains invalid scope '{scope}'.")