Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor SharedTokenCacheCredential exception handling #11962

Merged
merged 2 commits into from
Jun 10, 2020

Conversation

chlowell
Copy link
Member

This refactors the way the credential handles exceptions from MSAL to make it possible for get_token to raise ValueError when called with no scopes. Closes #11553.

@chlowell chlowell added Client This issue points to a problem in the data-plane of the library. Azure.Identity labels Jun 10, 2020
@chlowell chlowell requested a review from xiangyan99 June 10, 2020 21:45
@chlowell chlowell requested a review from schaabs as a code owner June 10, 2020 21:45
@chlowell chlowell merged commit 2614fa1 into Azure:master Jun 10, 2020
@chlowell chlowell deleted the refactor-exception-handling branch June 10, 2020 23:01
iscai-msft added a commit to iscai-msft/azure-sdk-for-python that referenced this pull request Jun 11, 2020
…into have_azure_key_credential_policy_take_kwargs

* 'master' of https://github.com/Azure/azure-sdk-for-python:
  typing fix (Azure#11985)
  remove test code that works around service bug (fixed) (Azure#11961)
  Refactor SharedTokenCacheCredential exception handling (Azure#11962)
  add the previous versions of azure-servicebus to the regression exclusion list (Azure#11957)
  Increment package version after release of azure_ai_formrecognizer (Azure#11954)
  Remove redundant await_prepared_test (Azure#11930)
  update changelog date (Azure#11934)
  Sync eng/common directory with azure-sdk-tools repository (Azure#11925)
  [formrecognizer] update recordings, add delay, switch to westcentralus region (Azure#11916)
  Increment package version after release of azure_identity (Azure#11931)
  Resolve Breaking Aggregate Reports (Azure#11825)
  Increment package version after release of azure_ai_textanalytics (Azure#11928)
  update error msg if not supported (Azure#11915)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SharedTokenCacheCredential.get_token() should raise ValueError
2 participants