[configtelemetry] Add a read header timeout of 10s on the configtelemetry Prometheus HTTP server

[atoulme]

Description

Add 10s read header timeout on the configtelemetry Prometheus HTTP server.

Link to tracking issue

Fixes #5699

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.64%. Comparing base (88b3153) to head (f095848).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #10675   +/-   ##
=======================================
  Coverage   91.64%   91.64%           
=======================================
  Files         406      406           
  Lines       18998    18999    +1     
=======================================
+ Hits        17411    17412    +1     
  Misses       1227     1227           
  Partials      360      360           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[atoulme]

@codeboten given that you created the issue, please take a look?

[codeboten]

This is probably ok, but it's not solving the original issue correct? the places in the code that were triggering the lint failure are still unchanged?

[atoulme]

no this solves one of the lint failures, and the others are plugged via confighttp changes.

[atoulme]

See this commit and change for the other place where this occurs: https://github.com/open-telemetry/opentelemetry-collector/pull/10275/files#diff-1fadfcd3b4194004717150126d27e8a32fdaa8292ca99e81a937d26abe0998faR351

[codeboten]

Thanks @atoulme, can G112 be removed then? https://github.com/open-telemetry/opentelemetry-collector/pull/5700/files

[atoulme]

@codeboten it appears so! I have reverted the commit and added it to the PR.

[codeboten]

Thanks!!