Add support for setting server_name as query parameter

[johanfleury]

Implements #74

[ribbybibby]

The same proposal has been rejected in the blackbox_exporter: prometheus/blackbox_exporter#624.

I'm sympathetic to the reasoning there that this issue can be solved by configuration management.

[k0ste]

I agree that this possible to cover cases like this via IaC, in per-module manner. We do it like this with ssl_exporter Ansible role

[johanfleury]

I’m closing the PR as I’m not using ssl_exporter that way anymore since I left my previous company.

I still think that having to manage a configuration file is a pain, especially when you already have everything you need in the service discovery labels.

Of course everything is always possible (depending on the time you’re willing to put into it), but managing the configuration dynamically in the Kubernetes context (in which this issue arose) is not the simplest thing to do.

[ldaneliukas]

@ribbybibby seeing as blackbox_exporter now accepts hostname as a parameter, can this be re-opened?