blackbox_exporter icon indicating copy to clipboard operation
blackbox_exporter copied to clipboard

Add hostname param support for tls servername

Open BoringCat opened this issue 2 years ago • 1 comments

Modified for this case

  • need to monitor multiple domain which is dynamic resolve to different instance
  • instance handle multiple certificate and distinguish with SNI

To monitor more than thousands domain. I can not just create config for prometheus and blackbox-exporter dynamiced. If each domain need it own job and module. The config file will to large and may be affect prometheus performance.

So it will be perfect if single module can handle all request just with hostname param like http probe 👍

BoringCat avatar Jul 14 '22 03:07 BoringCat

Thanks. This would require tests.

roidelapluie avatar Jul 14 '22 07:07 roidelapluie

fyi: https://github.com/prometheus/blackbox_exporter/pull/981 is also fixing the same issue.

electron0zero avatar Nov 15 '22 19:11 electron0zero

fyi: #981 is also fixing the same issue.

Well, after i make this PR. My target was changed. I need to known more infomation about the certificate. I'm now creating my own exporter. So I have no time work for this. If #981 can fix it and pass test. Please close this.

BoringCat avatar Nov 16 '22 01:11 BoringCat

fyi: #981 is also fixing the same issue.

Well, after i make this PR. My target was changed. I need to known more infomation about the certificate. I'm now creating my own exporter. So I have no time work for this. If #981 can fix it and pass test. Please close this.

closing in favor of #981

electron0zero avatar Dec 23 '22 05:12 electron0zero