Implemented Last-Modified HTTP header metric #407
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
joker234
force-pushed
the
http-last-modified
branch
from
41b5366
to
0916ab8
Compare
This feature returns a probe_http_last_modified metric with the content of the Last-Modified HTTP response header, if it is set by the server. Signed-off-by: Johannes Visintini <[email protected]>
joker234
force-pushed
the
http-last-modified
branch
from
a5509cb
to
80cfb20
Compare
|
Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This feature returns a
probe_http_last_modifiedmetric with the content of the Last-Modified HTTP response header, if it is set by the server.That feature would be nice for (blackbox) monitoring, if a service that should regularly update a website, is working or not.
P.S.: someone encountered me to the fact, that registering the gauges shouldn't be repeated in each probe, but only at initializing the exporter. This isn't part of this PR.
blackbox_exporter/prober/http.go
Lines 181 to 187 in 22be143