-
Notifications
You must be signed in to change notification settings - Fork 255
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
Error handling refactor #1539
Comments
@danielnelson here is one of the error sites in the code: https://github.com/influxdata/chronograf/blob/master/influx/databases.go#L25 We'll need to run the escaping algorithm used in Influxql here: https://github.com/influxdata/influxdb/blob/master/influxql/parser.go#L2930-L2933 |
• Chose one common error reporting scheme See also #1301 |
Steps:
oh"no
Expected:
Database is created, or error displayed indicating the database name is invalid.
Actual:
Page displays "Cannot communicate with server."
Server logs contain:
The text was updated successfully, but these errors were encountered: