Skip to content

Refactor SharedTokenCacheCredential exception handling#11962

Merged
chlowell merged 2 commits intoAzure:masterfrom
chlowell:refactor-exception-handling
Jun 10, 2020
Merged

Refactor SharedTokenCacheCredential exception handling#11962
chlowell merged 2 commits intoAzure:masterfrom
chlowell:refactor-exception-handling

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