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

Enable MSAL logging #19637

Merged
merged 2 commits into from
Mar 26, 2021
Merged

Conversation

christothes
Copy link
Member

@christothes christothes commented Mar 19, 2021

Wires up MSAL logging through the AzureIdentityEventSource

fixes #14450

@christothes christothes requested a review from schaabs as a code owner March 19, 2021 14:58
@ghost ghost added the Azure.Identity label Mar 19, 2021
@christothes christothes self-assigned this Mar 19, 2021
@christothes christothes added the Client This issue points to a problem in the data-plane of the library. label Mar 19, 2021
@christothes
Copy link
Member Author

/azp run net - identity - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@christothes christothes force-pushed the users/chriss/logMSal branch from ef06ff1 to 6f9efbc Compare March 26, 2021 19:07
@christothes christothes enabled auto-merge (squash) March 26, 2021 19:13
@christothes christothes merged commit 845a37d into Azure:master Mar 26, 2021
@ayeltsov
Copy link

ayeltsov commented Mar 26, 2021

Admittedly I took very brief look at the PR, so may be missing something obvious, but how does one enable and/or control level of the logging?
Should this change be accompanied by any documentation/sample follow-ups ?

@christothes
Copy link
Member Author

Admittedly I took very brief look at the PR, so may be missing something obvious, but how does one enable and/or control level of the logging?
Should this change be accompanied by any documentation/sample follow-ups ?

This is configured the same way you'd set ETL logging levels generally. Here is an example: https://docs.microsoft.com/en-us/dotnet/azure/logging#configure-custom-logging

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.

[FEATURE REQ]Azure.Identity should expose trace from MSAL library
3 participants