feat: allow disabling otel exporter #1436
Open
+19
−12
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
🦋 Changeset detectedLatest commit: 36d4dd0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
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 |
|
@Misfits09 is attempting to deploy a commit to the HyperDX Team on Vercel. A member of the Team first needs to authorize it. |
Misfits09
force-pushed
the
feature/allow-disabling-otel-exporter
branch
2 times, most recently
from
b8cf975 to
60f179a
Compare
Misfits09
force-pushed
the
feature/allow-disabling-otel-exporter
branch
from
60f179a to
36d4dd0
Compare
wrn14897
reviewed
Dec 31, 2025
Comment on lines
+11
to
12
| exporterEnabled: HDX_EXPORTER_ENABLED, | ||
| apiKey: HDX_API_KEY, |
Member
There was a problem hiding this comment.
I’d suggest removing the apiKey when HDX_EXPORTER_ENABLED is true. In this way users don’t need to inject an env var on the frontend side and rebuild the image
wrn14897
reviewed
Dec 31, 2025
Comment on lines
+29
to
+33
| ARG OTEL_EXPORTER_ENABLED | ||
| ARG OTEL_SERVICE_NAME | ||
| ARG IS_LOCAL_MODE | ||
| ENV NEXT_PUBLIC_OTEL_EXPORTER_OTLP_ENDPOINT $OTEL_EXPORTER_OTLP_ENDPOINT | ||
| ENV NEXT_PUBLIC_OTEL_EXPORTER_ENABLED $OTEL_EXPORTER_ENABLED |
Member
There was a problem hiding this comment.
No changes are needed here if the env var is only available on the next api side
Member
|
@Misfits09 Lmk if comments make sense. Would love to get this PR out |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
The default behavior to try to export logs/metrics/traces to an otel collector is causing issues in cases where this collector is not publicly accessible. Therefore adding an option to disable it would avoid having many errors in the Network tabs of the browser.
It has been briefly discussed here on discord
Changes
Added a new
HDX_EXPORTER_ENABLEDconfiguration variable that prevents the instanciation ofHyperDX.init()from@hyperdx/browser. Which disables sending metrics to the collector