Validate configuration after loading
Review Request #10006 — Created June 7, 2018 and submitted — Latest diff uploaded
We now validate the parsed
config.jsonand check for any omitted
fields. Go doesn't let us error on a missing field during
json.Unmarshallso we have to check for its 0 value (e.g.,""for
strings). We now return an error if any required field is missing. In
the case of a missing Port field, we default to 8888 and log a warning
message.Additionally, all files specfied in
config.jsonare resolve relative
to the path of the config file unless they are absolute paths.
Testing done:
Rango fmt ./....
Rango test ./....