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 JSON parsing error messaging #2

Open
dougbtv opened this issue Dec 19, 2018 · 1 comment
Open

Improved JSON parsing error messaging #2

dougbtv opened this issue Dec 19, 2018 · 1 comment

Comments

@dougbtv
Copy link
Member

dougbtv commented Dec 19, 2018

Feng mentioned during a recent demo that one thing that might not be intuitive to users is how to fix an error that's caused by a JSON parsing error, with an error message such as:

error parsing configuration: invalid character ':' after top-level value

Two thoughts regarding this:

  1. We could potentially wrap / trap this error and give a more general message, maybe something along the lines of "JSON parsing error in config: field, please check that there's valid JSON within"
  2. We could go upstream and update the CNI libraries to provide some better messaging.
@dcbw
Copy link
Member

dcbw commented Jan 3, 2019

@dougbtv we should certainly do #2 regardless

dougbtv added a commit that referenced this issue Oct 28, 2019
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

No branches or pull requests

2 participants