Add otel headers to health checks

[beefancohen]

Fixes #206

[changeset-bot]

🦋 Changeset detected

Latest commit: 4d252a9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@hyperdx/node-opentelemetry	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[brandon-pereira]

@wrn14897 I tested this locally and I see the OTEL_EXPORTER_OTLP_HEADERS headers being passed to the health check endpoint. I also added tests to help cover this case. Would love a second review on this if possible though!

[beefancohen]

@wrn14897 I tested this locally and I see the OTEL_EXPORTER_OTLP_HEADERS headers being passed to the health check endpoint. I also added tests to help cover this case. Would love a second review on this if possible though!

thanks for adding tests, i wasn't sure if we wanted to export the helper fn

[brandon-pereira]

@wrn14897 I tested this locally and I see the OTEL_EXPORTER_OTLP_HEADERS headers being passed to the health check endpoint. I also added tests to help cover this case. Would love a second review on this if possible though!

thanks for adding tests, i wasn't sure if we wanted to export the helper fn

Good point.. I moved the function to the utils folder :)

[brandon-pereira]

@beefancohen We will ship this in the next release - if you need it sooner, please let us know, and we can publish a patch version.

Thanks again for helping us improve the SDK! We really appreciate your input.

[wrn14897]

Thanks for the contribution

[beefancohen]

@beefancohen We will ship this in the next release - if you need it sooner, please let us know, and we can publish a patch version.

Thanks again for helping us improve the SDK! We really appreciate your input.

No rush it's not affecting any runtime behavior was just confusing me at first!