-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Create proper documentation on supported Browsers and Identity Providers #374
Comments
I don't work for Microsoft but I do work for a big retailer that is injecting B2C into their commerce platform and I will say the documentation for MSAL is little sparse. But having said that B2C is much more that MSAL and our decision to step into implementing B2C didn't even factor in the MSAL library. But you are correct. The documentation for MSAL needs to be stepped up. Working examples with different frameworks, etc... |
BTW, When you say Identity Providers... From my understanding... With B2C the JWT token you receive back is always from AAD. When you setup your B2C tenant and ask it to auth with Facebook you won't receive a Facebook JWT token. Even though you see a Facebook authentication prompt. |
I just read #177. Interesting. I'll have to test this when I get in the office. Is there a B2C cors setting that needs to be set? |
@visualjeff thanks for your comments. For us MSAL was a requirement since we are using vuejs SPA for our UI. I actually converted our app to hellojs and that works much better with multiple browsers. I hope MS improves documentation so that others don't have to waste time to see if current library fits their use case. |
Agreed. Microsoft needs a working example and test suite for each Javascript UI framework. I've been trying to integrate MSAL into ember.js. I have had some success but its an up hill climb. |
@visualjeff vladkosarev we have released the fix for IE and edge browser in MSAL 0.2.2. Please check the release notes for more details. |
@vladbarosan FYI. |
MSAL readme says - 'This library is suitable for use in a production environment. ' and since b2c has been GA for a long time that comes with a set of assumptions that don't seem to be true at all.
Current documentation is very confusing. For example https://github.com/AzureAD/microsoft-authentication-library-for-js/wiki/Using-msal.js-with-Internet-Explorer says 'This just works in production' which is obviously not the case according to #347. From what I can gather only Chrome and Firefox might be currently supported but that is not mentioned anywhere.
Identity Providers support seems to also be only partially working and there's no table to see what works and what doesn't. In #352 I mention Twitter not working but that's just what we tried. There's no way right now to know which providers are supported and which aren't.
All of this creates HUGE confusion when researching B2C as a viable solution. It is sold as GA and ready for production but it doesn't look like that's the case if MSAL is involved.
Is it possible to create documentation on supported Browsers and Identity Providers and put it right in README so that it is obvious to the customer if they should consider b2c at all? This would save your clients a ton of time.
Thanks.
The text was updated successfully, but these errors were encountered: