Skip to content

Better MSAL support for additional scenarios #38122

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

Closed
javiercn opened this issue Nov 5, 2021 · 1 comment
Closed

Better MSAL support for additional scenarios #38122

javiercn opened this issue Nov 5, 2021 · 1 comment
Assignees
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly feature-blazor-wasm-auth Priority:1 Work that is critical for the release, but we could probably ship without
Milestone

Comments

@javiercn
Copy link
Member

javiercn commented Nov 5, 2021

One of the most common complaints is the ability for people to pass in additional arguments to the underlying MSAL library that we don't expose ourselves. Instead of us having to add support for every single parameter, we might want to give people the "keys to the kingdom" and offer a lower level experience where they get to customize everything.

We would keep our library around, but we would "scaffold" the msal code into the templates (like we did for Azure AD/B2C) in previous versions and that would enable people to customize, update and interface with msal however they deemed fit. It doesn't pose a big burden on our side, as we would still need to update the library periodically, and we don't really care if that change is on the templates or on a nuget package. The main difference here would be that customers would be responsible for keeping the JS version of the library up to date (same as with a SPA) but will gain the ability to add features as needed without waiting for us to support them.

@javiercn javiercn added area-blazor Includes: Blazor, Razor Components feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly feature-blazor-wasm-auth jcn-p0 labels Nov 5, 2021
@javiercn javiercn added this to the .NET 7 Planning milestone Nov 5, 2021
@mkArtakMSFT mkArtakMSFT added triaged Priority:1 Work that is critical for the release, but we could probably ship without labels Nov 11, 2021
@mkArtakMSFT
Copy link
Member

@javiercn should this be moved to RC1?

@javiercn javiercn modified the milestones: 7.0-preview7, 7.0-rc1 Jul 18, 2022
@danroth27 danroth27 added the enhancement This issue represents an ask for new feature or an enhancement to an existing one label Aug 24, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Sep 23, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-blazor Includes: Blazor, Razor Components enhancement This issue represents an ask for new feature or an enhancement to an existing one feature-blazor-wasm This issue is related to and / or impacts Blazor WebAssembly feature-blazor-wasm-auth Priority:1 Work that is critical for the release, but we could probably ship without
Projects
None yet
Development

No branches or pull requests

3 participants