-
Notifications
You must be signed in to change notification settings - Fork 286
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
Implement public client application global cache #770
Conversation
03e8907
to
0a4d35c
Compare
0a4d35c
to
f2ba8a3
Compare
Though the testing for this requires human interaction, can we have some comments or sample code somewhere about how to validate it manually? |
Assume that there are two active tokens with different access to two databases; The first token with index 0 in the accounts has access to |
The scope of the token is database.windows.net, not the specific database. So using the same token for two databases will be fine. The token identifies the user to the database and database ACL settings determine whether that user is authorized to access the database. There really should never be two tokens in the situation you described. The first token would have been provided when the second was requested from MSAL. |
...crosoft.Data.SqlClient/src/Microsoft/Data/SqlClient/ActiveDirectoryAuthenticationProvider.cs
Outdated
Show resolved
Hide resolved
316ea44
to
f2ba8a3
Compare
Fixes an issue where driver continues to prompt for credentials when requesting access token for AAD authentication, mainly visible when using Interactive authentication.