Expose HTTPHost property in HTTPClientConfig

[jkroepke]

While I was integrate that feature in grafana/alloy#698, it figure out that the Host property was removed in #597. Turns out that I forget something.

Since host is problematic with DockerSD, i choice the term http_host to follow the pattern from http_headers

[bboreham]

Turns out that I forget something.

Please explain this further.

[bboreham]

In #549 you added Host to HTTPClientConfig and host to httpClientOptions.

Since only the latter was used, I removed the former. Now you adding it back, and hopefully using it.
Do we need both? Why?

[jkroepke]

Hey @bboreham,

I only need the property in HTTPClientConfig within the context of Grafana Alloy.

I have to apologize. When I started with Golang a year ago, I implemented the wrong side due to my lack of knowledge and missing tooling. I just replicated the userAgent logic, which misled me.

Looking at it now, I realize that the host property in httpClientOptions is incorrect.

If you agree, I can revert the host in httpClientOptions.

[bboreham]

Thanks for the explanation. It's fine, everybody makes mistakes. Just easier for me to follow if I can see the path.

However we may prefer to deprecate the un-needed one rather than removing it immediately.
That way we can see if anyone else came to depend on it.

[KarstenSiemer]

Hi! Anything missing here?

[SuperQ]

Ping @jkroepke @bboreham, any issues to resolve here?

[jkroepke]

@SuperQ from my understanding, the review asks for a more specific doc note which I applied now.