Skip to content

Pass the error message from MSAL when request fails#3346

Merged
neha-bhargava merged 6 commits into
masterfrom
nebharg/passMsalException
Apr 29, 2025
Merged

Pass the error message from MSAL when request fails#3346
neha-bhargava merged 6 commits into
masterfrom
nebharg/passMsalException

Conversation

@neha-bhargava
Copy link
Copy Markdown
Contributor

Propagate MSAL exception message

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)

Description

Pass exception message from MSAL to the user when the request fails

Fixes #3345

@neha-bhargava neha-bhargava requested a review from a team as a code owner April 28, 2025 20:24
Copy link
Copy Markdown
Collaborator

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread src/Microsoft.Identity.Web.TokenAcquisition/TokenAcquisition.cs
@neha-bhargava neha-bhargava merged commit 76db1e5 into master Apr 29, 2025
5 checks passed
@neha-bhargava neha-bhargava deleted the nebharg/passMsalException branch April 29, 2025 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AcquireTokenForConfidentialClient does not pass MSAL exception

4 participants