-
Notifications
You must be signed in to change notification settings - Fork 341
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
exporter: link to actual metricsPath #99
Conversation
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exporter.go
Outdated
<head><title>NGINX Exporter</title></head> | ||
<body> | ||
_, err := fmt.Fprintf(w, `<!DOCTYPE html> | ||
<meta charset="utf-8"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure if meta is neccesary, because the exporter already includes the charset in the response header:
$ curl http://localhost:9113/ -I
HTTP/1.1 200 OK
. . .
Content-Type: text/html; charset=utf-8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, I've removed it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
@greut thanks! |
Proposed changes
The HTML on the
/
path show an hardcoded value for the metrics path. This fixes it.Checklist
Before creating a PR, run through this checklist and mark each as complete.