-
Notifications
You must be signed in to change notification settings - Fork 97
Open
Description
All fields , (e.g. in Config, TLSConfig) should have yaml:"<NAME>,omitempty"
https://github.com/prometheus/exporter-toolkit/blob/master/web/tls_config.go#L39
For example, it doesn't make sense that TLSConfig.MinVersion appears as 0 when marshaled to YAML. Prometheus crashes if you try to use this web config via --web.config.file=:
caller=main.go:949 level=error msg="Unable to validate web configuration file" err="unknown TLS version: 0"
Metadata
Metadata
Assignees
Labels
No labels