Skip to content
This repository has been archived by the owner on Oct 14, 2024. It is now read-only.

Commit

Permalink
Merge pull request #416 from microsoft/fix/cae-parameter
Browse files Browse the repository at this point in the history
fix: wrong cae parameter name for azure identity
  • Loading branch information
baywet authored Aug 21, 2024
2 parents 84c3d9e + 6b08806 commit a59a85a
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -103,12 +103,12 @@ async def get_authorization_token(
result = self._credentials.get_token(
*self._scopes,
claims=decoded_claim,
is_cae_enabled=self._is_cae_enabled,
enable_cae=self._is_cae_enabled,
**self._options
)
else:
result = self._credentials.get_token(
*self._scopes, claims=decoded_claim, is_cae_enabled=self._is_cae_enabled
*self._scopes, claims=decoded_claim, enable_cae=self._is_cae_enabled
)

if inspect.isawaitable(result):
Expand Down
2 changes: 1 addition & 1 deletion tests/test_azure_identity_authentication_provider.py
Original file line number Diff line number Diff line change
Expand Up @@ -42,5 +42,5 @@ async def test_adds_claim_to_the_token_context(mocker):
credential.get_token.assert_called_once_with(
'https://graph.microsoft.com/.default',
claims="""{"access_token":{"nbf":{"essential":true, "value":"1652813508"}}}""",
is_cae_enabled=True
enable_cae=True
)

0 comments on commit a59a85a

Please sign in to comment.