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

Add Graph access to the BWA+OIDC article+samples #33099

Closed
guardrex opened this issue Jul 16, 2024 · 3 comments
Closed

Add Graph access to the BWA+OIDC article+samples #33099

guardrex opened this issue Jul 16, 2024 · 3 comments

Comments

@guardrex
Copy link
Collaborator

guardrex commented Jul 16, 2024

Description

Current ASP.NET Core guidance in the Graph docs (along with the weather data guidance) indirectly address how Graph would be added to a BWA with Interactive Auto/WASM rendering. Similar to what we did for Graph+Blazor WASM, it would be quicker and easier for devs to implement Graph if we show them how to do it in our docs. Forwarding devs to the Graph docs is going to be too challenging for new devs and new-to-Blazor devs and will end in tears 😢 for many ... and issues for us 💰.

UPDATE: Given that the current section on app roles with ME-ID looks like it may lead to cookie/header length problems, let's place Graph SDK/API for AD groups and AD built-in Admin Roles ... and let that be the example of using Graph.

UPDATE: Actually, I suppose the current section would be fine if users only have a couple of AD groups/roles. Perhaps, there should be a remark to that effect for the existing section ... and then there should be a new section for Graph SDK/API for AD groups/roles. Then, devs can decide which approach makes the most sense for an organization. Still tho, I think we'd favor telling devs to adopt the Graph SDK/API to harden the app for the future. 🤔 ... I'll think it over next week.

Working this will now solve #33450.

Page URL

https://learn.microsoft.com/en-us/aspnet/core/blazor/security/blazor-web-app-with-oidc?view=aspnetcore-8.0

Content source URL

https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/blazor/security/blazor-web-app-with-oidc.md

Document ID

c3346d3a-346b-8db4-2650-ec044b3f0dd9

Article author

@guardrex

@guardrex
Copy link
Collaborator Author

guardrex commented Sep 3, 2024

We're not going to pursue this at this time. We'll add Graph guidance for the upcoming BWA+MS Identity Web sample and article.

@guardrex guardrex closed this as completed Sep 3, 2024
@guardrex
Copy link
Collaborator Author

guardrex commented Sep 4, 2024

Trying one more time this morning using a different approach.

@guardrex guardrex reopened this Sep 4, 2024
@guardrex
Copy link
Collaborator Author

guardrex commented Sep 4, 2024

RE-closing because I could only get the REST API to play nicely with the BWA+OIDC app. We won't cover it beyond passing the claims in the auth cookie at this time.

@guardrex guardrex closed this as completed Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

2 participants