-
Notifications
You must be signed in to change notification settings - Fork 804
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: prepare release 1.24.1/0.51.1 (backport bugfixes) #4682
chore: prepare release 1.24.1/0.51.1 (backport bugfixes) #4682
Conversation
…urce -> ...) (open-telemetry#4653) * fix(resources): prevent circular import (resource -> detector -> resource -> ...) * fixup! fix(resources): prevent circular import (resource -> detector -> resource -> ...)
…efaults() when a process polyfill is used (open-telemetry#4649) * fix(core): align inconsistent behavior of getEnv() and getEnvWithoutDefaults() when a process polyfill is used * Update CHANGELOG.md
open-telemetry#4678) * fix(exporter-metrics-otlp-grpc) Add explicit otlp-exporter-base dependency to exporter-metrics-otlp-grpc * Update CHANGELOG.md --------- Co-authored-by: Marc Pichler <marc.pichler@dynatrace.com>
I reached out to Datadog about this through @Qard. As the PR currently stands it would break dd-trace which I would like to avoid. I think we should delay this for a short time in order to allow them to respond, but I don't want to delay it forever. |
@dyladan this is a PR that just backports bugfixes, I think you meant to comment over at #4677 🙂 |
Thanks |
41c2626
into
open-telemetry:release/1.24.x-0.51.x
Description
Backports bugfixes to 1.24.x and 0.51.x
Reason:
1.24.1
🐛 (Bug Fix)
getEnv()
andgetEnvWithoutDefaults()
when aprocess
polyfill is used #4648 @pichlermarcgetEnvWithoutDefaults()
would useprocess.env
if it was defined when running in a browser, whilegetEnv()
would always use_globalThis
. Now both use_globalThis
when running in a browser.@opentelemetry/resources
0.51.1
🐛 (Bug Fix)