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

Improved errors at startup for problematic config file #90

Open
bryanpedini opened this issue Sep 5, 2024 · 0 comments
Open

Improved errors at startup for problematic config file #90

bryanpedini opened this issue Sep 5, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@bryanpedini
Copy link
Contributor

As described a bit in #89, there are two "cryptical" errors shown with an empty or a missing config file, that dive deep into the code / traceback, presumably handled by Node itself.
I'm not a Node dev so I can't help on those, but perhaps it would be better to avoid confusion for end users, to catch problematic config files and display human readable errors instead.

I'm not saying to build a parser from scratch and give the line at which the error was found if somebody forgot a comma in the json, but perhaps a categorization of errors such divided:

  • "Config file missing" (or corrupted?), "unable to start"
  • "Config file is empty, unable to start"
  • "Syntax error in config file, unable to start"

Or something similar...

@Ardakilic Ardakilic added the enhancement New feature or request label Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants