Skip to content

Conversation

@guardrex
Copy link
Collaborator

@guardrex guardrex commented Dec 9, 2024

Fixes #34235

Thanks @albi005! 🚀

Unless either of you wants to supply the actual code for what the NOTE suggests, we'll go with this, and it will be up to devs to write their own code for it.


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/security/blazor-web-app-with-oidc.md aspnetcore/blazor/security/blazor-web-app-with-oidc

@guardrex guardrex requested a review from halter73 December 9, 2024 20:31
Got it! Thanks!

Co-authored-by: Albert Ragány-Németh <[email protected]>
Copy link

@albi005 albi005 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code for the BFF part is missing.

@guardrex
Copy link
Collaborator Author

guardrex commented Dec 9, 2024

This is no longer in a pivot because I moved it down to your suggested location, so we only need one copy of the content.

@guardrex
Copy link
Collaborator Author

guardrex commented Dec 9, 2024

LGTM ... I'll go ahead and merge this.

@halter73, if you have any change suggestions, I'm 👂 here, and I'll put in a patch PR if needed. Otherwise, this is now tracked for .NET 10. I'll keep an 👁️ on it next year.

Thanks again for your help, @albi005!

@guardrex guardrex merged commit 6ae701a into main Dec 9, 2024
3 checks passed
@guardrex guardrex deleted the guardrex-patch-4 branch December 9, 2024 21:07
@guardrex guardrex removed the request for review from halter73 December 9, 2024 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BlazorWebAppOidc sample: incorrect CookieOidcRefresher implementation

3 participants