Fix HTTPClientConfig JSON marshalling #651

pracucci · 2024-06-07T14:24:50Z

I think #416 introduced an issue. json marshalling doesn't support inline and so when HTTPClientConfig gets marshalled to JSON it gets marshalled to something like this:

{"http_headers":{"Headers":{"Header-Name":{}}}

while I would expect something like this:

{"http_headers":{"Header-Name":{}}

This PR should fix it.

Signed-off-by: Marco Pracucci <marco@pracucci.com>

roidelapluie · 2024-06-07T15:06:54Z

Thanks!

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/prometheus/common](https://github.com/prometheus/common) | `v0.54.0` -> `v0.55.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>prometheus/common (github.com/prometheus/common)</summary> ### [`v0.55.0`](https://github.com/prometheus/common/releases/tag/v0.55.0) [Compare Source](https://github.com/prometheus/common/compare/v0.54.0...v0.55.0) #### What's Changed - Update Go modules by [@SuperQ](https://github.com/SuperQ) in [prometheus/common#643 - enable errcheck linter by [@mmorel-35](https://github.com/mmorel-35) in [prometheus/common#637 - Add a `RELEASE.md` and add [@gotjosh](https://github.com/gotjosh) as a mantainer by [@gotjosh](https://github.com/gotjosh) in [prometheus/common#644 - Move goautoneg to external dependency by [@mikelolasagasti](https://github.com/mikelolasagasti) in [prometheus/common#625 - Expose secret as SecretReader and InlineSecret from config package by [@pracucci](https://github.com/pracucci) in [prometheus/common#650 - Fix HTTPClientConfig JSON marshalling by [@pracucci](https://github.com/pracucci) in [prometheus/common#651 - Expose secret as FileSecret from config package by [@alanprot](https://github.com/alanprot) in [prometheus/common#653 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#646 - Set http_headers to be omit empty by [@yeya24](https://github.com/yeya24) in [prometheus/common#655 - chore: add HumanizeTimestamp; make ConvertToFloat exportable by [@freak12techno](https://github.com/freak12techno) in [prometheus/common#654 - Bump github.com/aws/aws-sdk-go from 1.53.14 to 1.54.7 in /sigv4 by [@dependabot](https://github.com/dependabot) in [prometheus/common#659 - Bump golang.org/x/oauth2 from 0.20.0 to 0.21.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#656 - Bump google.golang.org/protobuf from 1.34.1 to 1.34.2 by [@dependabot](https://github.com/dependabot) in [prometheus/common#658 - Bump golang.org/x/net from 0.25.0 to 0.26.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#657 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#660 - Add SigV4 FIPS STS endpoint config by [@rajagopalanand](https://github.com/rajagopalanand) in [prometheus/common#649 #### New Contributors - [@gotjosh](https://github.com/gotjosh) made their first contribution in [prometheus/common#644 - [@mikelolasagasti](https://github.com/mikelolasagasti) made their first contribution in [prometheus/common#625 - [@alanprot](https://github.com/alanprot) made their first contribution in [prometheus/common#653 - [@yeya24](https://github.com/yeya24) made their first contribution in [prometheus/common#655 - [@rajagopalanand](https://github.com/rajagopalanand) made their first contribution in [prometheus/common#649 **Full Changelog**: prometheus/common@v0.54.0...v0.55.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/jaegertracing/jaeger).  --------- Signed-off-by: Mend Renovate <bot@renovateapp.com> Signed-off-by: Yuri Shkuro <github@ysh.us> Co-authored-by: Yuri Shkuro <github@ysh.us>

…5806) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/prometheus/common](https://github.com/prometheus/common) | `v0.54.0` -> `v0.55.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>prometheus/common (github.com/prometheus/common)</summary> ### [`v0.55.0`](https://github.com/prometheus/common/releases/tag/v0.55.0) [Compare Source](https://github.com/prometheus/common/compare/v0.54.0...v0.55.0) #### What's Changed - Update Go modules by [@SuperQ](https://github.com/SuperQ) in [prometheus/common#643 - enable errcheck linter by [@mmorel-35](https://github.com/mmorel-35) in [prometheus/common#637 - Add a `RELEASE.md` and add [@gotjosh](https://github.com/gotjosh) as a mantainer by [@gotjosh](https://github.com/gotjosh) in [prometheus/common#644 - Move goautoneg to external dependency by [@mikelolasagasti](https://github.com/mikelolasagasti) in [prometheus/common#625 - Expose secret as SecretReader and InlineSecret from config package by [@pracucci](https://github.com/pracucci) in [prometheus/common#650 - Fix HTTPClientConfig JSON marshalling by [@pracucci](https://github.com/pracucci) in [prometheus/common#651 - Expose secret as FileSecret from config package by [@alanprot](https://github.com/alanprot) in [prometheus/common#653 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#646 - Set http_headers to be omit empty by [@yeya24](https://github.com/yeya24) in [prometheus/common#655 - chore: add HumanizeTimestamp; make ConvertToFloat exportable by [@freak12techno](https://github.com/freak12techno) in [prometheus/common#654 - Bump github.com/aws/aws-sdk-go from 1.53.14 to 1.54.7 in /sigv4 by [@dependabot](https://github.com/dependabot) in [prometheus/common#659 - Bump golang.org/x/oauth2 from 0.20.0 to 0.21.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#656 - Bump google.golang.org/protobuf from 1.34.1 to 1.34.2 by [@dependabot](https://github.com/dependabot) in [prometheus/common#658 - Bump golang.org/x/net from 0.25.0 to 0.26.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#657 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#660 - Add SigV4 FIPS STS endpoint config by [@rajagopalanand](https://github.com/rajagopalanand) in [prometheus/common#649 #### New Contributors - [@gotjosh](https://github.com/gotjosh) made their first contribution in [prometheus/common#644 - [@mikelolasagasti](https://github.com/mikelolasagasti) made their first contribution in [prometheus/common#625 - [@alanprot](https://github.com/alanprot) made their first contribution in [prometheus/common#653 - [@yeya24](https://github.com/yeya24) made their first contribution in [prometheus/common#655 - [@rajagopalanand](https://github.com/rajagopalanand) made their first contribution in [prometheus/common#649 **Full Changelog**: prometheus/common@v0.54.0...v0.55.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-go-contrib).  Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

…5552) [![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/prometheus/common](https://github.com/prometheus/common) | `v0.54.0` -> `v0.55.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>prometheus/common (github.com/prometheus/common)</summary> ### [`v0.55.0`](https://github.com/prometheus/common/releases/tag/v0.55.0) [Compare Source](https://github.com/prometheus/common/compare/v0.54.0...v0.55.0) #### What's Changed - Update Go modules by [@SuperQ](https://github.com/SuperQ) in [prometheus/common#643 - enable errcheck linter by [@mmorel-35](https://github.com/mmorel-35) in [prometheus/common#637 - Add a `RELEASE.md` and add [@gotjosh](https://github.com/gotjosh) as a mantainer by [@gotjosh](https://github.com/gotjosh) in [prometheus/common#644 - Move goautoneg to external dependency by [@mikelolasagasti](https://github.com/mikelolasagasti) in [prometheus/common#625 - Expose secret as SecretReader and InlineSecret from config package by [@pracucci](https://github.com/pracucci) in [prometheus/common#650 - Fix HTTPClientConfig JSON marshalling by [@pracucci](https://github.com/pracucci) in [prometheus/common#651 - Expose secret as FileSecret from config package by [@alanprot](https://github.com/alanprot) in [prometheus/common#653 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#646 - Set http_headers to be omit empty by [@yeya24](https://github.com/yeya24) in [prometheus/common#655 - chore: add HumanizeTimestamp; make ConvertToFloat exportable by [@freak12techno](https://github.com/freak12techno) in [prometheus/common#654 - Bump github.com/aws/aws-sdk-go from 1.53.14 to 1.54.7 in /sigv4 by [@dependabot](https://github.com/dependabot) in [prometheus/common#659 - Bump golang.org/x/oauth2 from 0.20.0 to 0.21.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#656 - Bump google.golang.org/protobuf from 1.34.1 to 1.34.2 by [@dependabot](https://github.com/dependabot) in [prometheus/common#658 - Bump golang.org/x/net from 0.25.0 to 0.26.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#657 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#660 - Add SigV4 FIPS STS endpoint config by [@rajagopalanand](https://github.com/rajagopalanand) in [prometheus/common#649 #### New Contributors - [@gotjosh](https://github.com/gotjosh) made their first contribution in [prometheus/common#644 - [@mikelolasagasti](https://github.com/mikelolasagasti) made their first contribution in [prometheus/common#625 - [@alanprot](https://github.com/alanprot) made their first contribution in [prometheus/common#653 - [@yeya24](https://github.com/yeya24) made their first contribution in [prometheus/common#655 - [@rajagopalanand](https://github.com/rajagopalanand) made their first contribution in [prometheus/common#649 **Full Changelog**: prometheus/common@v0.54.0...v0.55.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-go).  Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/prometheus/common](https://github.com/prometheus/common) | `v0.54.0` -> `v0.55.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>prometheus/common (github.com/prometheus/common)</summary> ### [`v0.55.0`](https://github.com/prometheus/common/releases/tag/v0.55.0) [Compare Source](https://github.com/prometheus/common/compare/v0.54.0...v0.55.0) #### What's Changed - Update Go modules by [@SuperQ](https://github.com/SuperQ) in [prometheus/common#643 - enable errcheck linter by [@mmorel-35](https://github.com/mmorel-35) in [prometheus/common#637 - Add a `RELEASE.md` and add [@gotjosh](https://github.com/gotjosh) as a mantainer by [@gotjosh](https://github.com/gotjosh) in [prometheus/common#644 - Move goautoneg to external dependency by [@mikelolasagasti](https://github.com/mikelolasagasti) in [prometheus/common#625 - Expose secret as SecretReader and InlineSecret from config package by [@pracucci](https://github.com/pracucci) in [prometheus/common#650 - Fix HTTPClientConfig JSON marshalling by [@pracucci](https://github.com/pracucci) in [prometheus/common#651 - Expose secret as FileSecret from config package by [@alanprot](https://github.com/alanprot) in [prometheus/common#653 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#646 - Set http_headers to be omit empty by [@yeya24](https://github.com/yeya24) in [prometheus/common#655 - chore: add HumanizeTimestamp; make ConvertToFloat exportable by [@freak12techno](https://github.com/freak12techno) in [prometheus/common#654 - Bump github.com/aws/aws-sdk-go from 1.53.14 to 1.54.7 in /sigv4 by [@dependabot](https://github.com/dependabot) in [prometheus/common#659 - Bump golang.org/x/oauth2 from 0.20.0 to 0.21.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#656 - Bump google.golang.org/protobuf from 1.34.1 to 1.34.2 by [@dependabot](https://github.com/dependabot) in [prometheus/common#658 - Bump golang.org/x/net from 0.25.0 to 0.26.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#657 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#660 - Add SigV4 FIPS STS endpoint config by [@rajagopalanand](https://github.com/rajagopalanand) in [prometheus/common#649 #### New Contributors - [@gotjosh](https://github.com/gotjosh) made their first contribution in [prometheus/common#644 - [@mikelolasagasti](https://github.com/mikelolasagasti) made their first contribution in [prometheus/common#625 - [@alanprot](https://github.com/alanprot) made their first contribution in [prometheus/common#653 - [@yeya24](https://github.com/yeya24) made their first contribution in [prometheus/common#655 - [@rajagopalanand](https://github.com/rajagopalanand) made their first contribution in [prometheus/common#649 **Full Changelog**: prometheus/common@v0.54.0...v0.55.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector-contrib).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [github.com/prometheus/common](https://github.com/prometheus/common) | `v0.54.0` -> `v0.55.0` | [![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fprometheus%2fcommon/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fprometheus%2fcommon/v0.54.0/v0.55.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>prometheus/common (github.com/prometheus/common)</summary> ### [`v0.55.0`](https://github.com/prometheus/common/releases/tag/v0.55.0) [Compare Source](https://github.com/prometheus/common/compare/v0.54.0...v0.55.0) ##### What's Changed - Update Go modules by [@SuperQ](https://github.com/SuperQ) in [prometheus/common#643 - enable errcheck linter by [@mmorel-35](https://github.com/mmorel-35) in [prometheus/common#637 - Add a `RELEASE.md` and add [@gotjosh](https://github.com/gotjosh) as a mantainer by [@gotjosh](https://github.com/gotjosh) in [prometheus/common#644 - Move goautoneg to external dependency by [@mikelolasagasti](https://github.com/mikelolasagasti) in [prometheus/common#625 - Expose secret as SecretReader and InlineSecret from config package by [@pracucci](https://github.com/pracucci) in [prometheus/common#650 - Fix HTTPClientConfig JSON marshalling by [@pracucci](https://github.com/pracucci) in [prometheus/common#651 - Expose secret as FileSecret from config package by [@alanprot](https://github.com/alanprot) in [prometheus/common#653 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#646 - Set http_headers to be omit empty by [@yeya24](https://github.com/yeya24) in [prometheus/common#655 - chore: add HumanizeTimestamp; make ConvertToFloat exportable by [@freak12techno](https://github.com/freak12techno) in [prometheus/common#654 - Bump github.com/aws/aws-sdk-go from 1.53.14 to 1.54.7 in /sigv4 by [@dependabot](https://github.com/dependabot) in [prometheus/common#659 - Bump golang.org/x/oauth2 from 0.20.0 to 0.21.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#656 - Bump google.golang.org/protobuf from 1.34.1 to 1.34.2 by [@dependabot](https://github.com/dependabot) in [prometheus/common#658 - Bump golang.org/x/net from 0.25.0 to 0.26.0 by [@dependabot](https://github.com/dependabot) in [prometheus/common#657 - Synchronize common files from prometheus/prometheus by [@prombot](https://github.com/prombot) in [prometheus/common#660 - Add SigV4 FIPS STS endpoint config by [@rajagopalanand](https://github.com/rajagopalanand) in [prometheus/common#649 ##### New Contributors - [@gotjosh](https://github.com/gotjosh) made their first contribution in [prometheus/common#644 - [@mikelolasagasti](https://github.com/mikelolasagasti) made their first contribution in [prometheus/common#625 - [@alanprot](https://github.com/alanprot) made their first contribution in [prometheus/common#653 - [@yeya24](https://github.com/yeya24) made their first contribution in [prometheus/common#655 - [@rajagopalanand](https://github.com/rajagopalanand) made their first contribution in [prometheus/common#649 **Full Changelog**: prometheus/common@v0.54.0...v0.55.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on tuesday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/open-telemetry/opentelemetry-collector).  --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: opentelemetrybot <107717825+opentelemetrybot@users.noreply.github.com>

Show HTTPClientConfig JSON marshalling issue

0f4f649

Signed-off-by: Marco Pracucci <marco@pracucci.com>

pracucci mentioned this pull request Jun 7, 2024

Add HTTP headers support to common HTTP client. #416

Merged

pracucci marked this pull request as draft June 7, 2024 14:25

pracucci mentioned this pull request Jun 7, 2024

Update vendored mimir-prometheus grafana/mimir#8305

Merged

4 tasks

Fix JSON marshalling

d310c4a

Signed-off-by: Marco Pracucci <marco@pracucci.com>

pracucci changed the title ~~Show HTTPClientConfig JSON marshalling issue~~ Fix HTTPClientConfig JSON marshalling Jun 7, 2024

pracucci marked this pull request as ready for review June 7, 2024 14:35

roidelapluie approved these changes Jun 7, 2024

View reviewed changes

roidelapluie merged commit 3183099 into prometheus:main Jun 7, 2024
7 checks passed

pracucci deleted the show-http-header-issues branch June 10, 2024 06:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix HTTPClientConfig JSON marshalling #651

Fix HTTPClientConfig JSON marshalling #651

pracucci commented Jun 7, 2024 •

edited

Loading

roidelapluie commented Jun 7, 2024

Fix HTTPClientConfig JSON marshalling #651

Fix HTTPClientConfig JSON marshalling #651

Conversation

pracucci commented Jun 7, 2024 • edited Loading

roidelapluie commented Jun 7, 2024

pracucci commented Jun 7, 2024 •

edited

Loading