-
Notifications
You must be signed in to change notification settings - Fork 31
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
Add msal browser kiota authentication provider #186
Comments
@sebastienlevert to evaluate whether we need that for preview or GA or post GA and prioritize it accordingly. |
Definitely POST-GA. I think Azure Identity should be good enough for now. |
I implemented something like this in a react app that may be related https://gist.github.com/lroos/0e0cec0346803368b3268bdae8a85101; it's a TokenCredential type that I passed into AzureIdentityAuthenticationProvider. I just had to set the allowedHosts to my endpoint origin. This on the right track? |
Hi @lroos This could work, but adds another additional level of indirection that is not needed. The suggestion here was instead to create a new package, that DOES NOT have a dependency on azure core, but instead on MSAL, and implement both BaseBearerAuthenticationProvider and IAccessTokenProvider with the primitive MSAL is using : ClientApplication Is this something you'd like to submit a pull request for provided some guidance? |
No description provided.
The text was updated successfully, but these errors were encountered: