Skip to content

Make it simpler for customers to investigate Blazor Auth issues #39412

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
mkArtakMSFT opened this issue Jan 10, 2022 · 10 comments
Closed

Make it simpler for customers to investigate Blazor Auth issues #39412

mkArtakMSFT opened this issue Jan 10, 2022 · 10 comments
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-server-auth feature-blazor-wasm-auth
Milestone

Comments

@mkArtakMSFT
Copy link
Contributor

mkArtakMSFT commented Jan 10, 2022

There have been many issues customers run into, where it's hard to understand what is the underlying issue. Whether it's a configuration issue or framework issue. This issue will be used to track the work for simplifying such investigations.

@mkArtakMSFT mkArtakMSFT added enhancement This issue represents an ask for new feature or an enhancement to an existing one area-blazor Includes: Blazor, Razor Components feature-blazor-server-auth feature-blazor-wasm-auth labels Jan 10, 2022
@mkArtakMSFT mkArtakMSFT added this to the .NET 7 Planning milestone Jan 10, 2022
@ghost
Copy link

ghost commented Jan 10, 2022

Thanks for contacting us.

We're moving this issue to the .NET 7 Planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.

@Lukazoid
Copy link

Lukazoid commented Feb 1, 2022

I feel like even allowing setup of the oidc-client logger would help a lot.

Currently I have a token endpoint returning a valid access_token and id_token but the UI simply says: "There was an error trying to log you in: ''", with nothing in the console or VS output window, really not sure how to investigate this further.

@andrew-tevent
Copy link

Agree, AuthenticationService.js should have config options which allow the Oidc.Log.Logger = console and Oidc.Log.LogLevel = ?. This would at least be a start. It's a pain that you have to build a custom version of AuthenticationService to turn the OIDC logging on.

@mkArtakMSFT
Copy link
Contributor Author

Moving to the RC1 milestone, as @javiercn has some work on hand which is going to improve this. Let's close this issue when the related PRs get merged.

@javiercn
Copy link
Member

We did add logging support on the JS libraries which covers this

@oatsoda
Copy link

oatsoda commented Aug 30, 2022

@javiercn Is there an example anywhere as to what we can configure?

@javiercn
Copy link
Member

@oatsoda not yet, it was added in RC1 which has not been released yet.

@mkArtakMSFT
Copy link
Contributor Author

We did add logging support on the JS libraries which covers this

@javiercn will these logs show up as part of normal / default ASP.NET Core app logs, or is there any special configuration required to enable them?

@guardrex
Copy link
Contributor

guardrex commented Sep 29, 2022

@javiercn ... Is there anything to document for 7.0 on this one? There's no cross-linked PR for me to research further.

UPDATE (10/3): Poked around a bit more, and the issues have circular references and don't seem to point to any particular PR for this ...

Looks now like it was part of #43954.

@javiercn
Copy link
Member

javiercn commented Oct 3, 2022

@guardrex turning the logging for Microsoft.AspNetCore.Components.WebAssembly.Authentication to debug or trace will log additional information on the browser console to help troubleshoot issues.

@ghost ghost locked as resolved and limited conversation to collaborators Nov 2, 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-server-auth feature-blazor-wasm-auth
Projects
None yet
Development

No branches or pull requests

6 participants