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

Make dashboard print a good error message when configuration is invalid #3168

Closed
JamesNK opened this issue Mar 26, 2024 · 3 comments · Fixed by #3243
Closed

Make dashboard print a good error message when configuration is invalid #3168

JamesNK opened this issue Mar 26, 2024 · 3 comments · Fixed by #3243

Comments

@JamesNK
Copy link
Member

JamesNK commented Mar 26, 2024

See #3119 (comment)

@leslierichardson95
Copy link

@JamesNK what configuration are you referring to here? Is it authentication related?

@JamesNK
Copy link
Member Author

JamesNK commented Mar 28, 2024

@KalleOlaviNiemitalo
Copy link

Startup error reporting has been problematic in my applications as well. I wonder if OptionsValidationException should be changed to override ToString() or Message to include OptionsType and OptionsName in addition to the failure messages; then those would be part of ILogger exception logging with no effort from applications. (If the options are bound to configuration, then it would be even better to include the configuration path in the log entry, but that looks much more difficult to implement.)

@github-actions github-actions bot locked and limited conversation to collaborators May 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants