-
Notifications
You must be signed in to change notification settings - Fork 207
Closed
Labels
Description
Describe the bug
When using the acquire_token_interactive
method with a timeout that is hit, the exception that is raised does not indicate a timeout occurred. Instead an AssertionError is raised.
To Reproduce
Code to reproduce:
app = msal.PublicClientApplication(
client_id=client_id, authority="https://{}/{}".format(AzureAuthorityHosts.AZURE_PUBLIC_CLOUD, tenant_id)
)
scope = "https://management.azure.com/.default"
result = app.acquire_token_interactive([scope], timeout=5)
Expected behavior
Some type of exception or message that is surfaced to the user, indicating a timeout has occurred.
What you see instead
File "/home/pvaneck/dev/pvaneck/azure-sdk-for-python/sdk/identity/azure-identity/test2.py", line 61, in <module>
result = app.acquire_token_interactive([scope], timeout=5)
File "/home/pvaneck/dev/pvaneck/azure-sdk-for-python/sdk/identity/azure-identity/.venv/lib/python3.10/site-packages/msal/application.py", line 1828, in acquire_token_interactive
response = _clean_up(self.client.obtain_token_by_browser(
File "/home/pvaneck/dev/pvaneck/azure-sdk-for-python/sdk/identity/azure-identity/.venv/lib/python3.10/site-packages/msal/oauth2cli/oidc.py", line 281, in obtain_token_by_browser
return super(Client, self).obtain_token_by_browser(
File "/home/pvaneck/dev/pvaneck/azure-sdk-for-python/sdk/identity/azure-identity/.venv/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 639, in obtain_token_by_browser
return self._obtain_token_by_browser(
File "/home/pvaneck/dev/pvaneck/azure-sdk-for-python/sdk/identity/azure-identity/.venv/lib/python3.10/site-packages/msal/oauth2cli/oauth2.py", line 677, in _obtain_token_by_browser
return self.obtain_token_by_auth_code_flow(
File "/home/pvaneck/dev/pvaneck/azure-sdk-for-python/sdk/identity/azure-identity/.venv/lib/python3.10/site-packages/msal/application.py", line 134, in obtain_token_by_auth_code_flow
assert isinstance(auth_code_flow, dict) and isinstance(auth_response, dict)
AssertionError
The MSAL Python version you are using
1.20.0