-
Notifications
You must be signed in to change notification settings - Fork 344
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
record network request bodies and headers in trace #8551
record network request bodies and headers in trace #8551
Conversation
🦋 Changeset detectedLatest commit: 9b015e1 The changes in this PR will be included in the next version bump. 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 💥 An error occurred when fetching the changed packages and changesets in this PR
|
c985b27
to
7a78ae9
Compare
3fa78a3
to
9b015e1
Compare
import('@highlight-run/client').then(async ({ Highlight }) => { | ||
highlight_obj = new Highlight( | ||
client_options, | ||
first_load_listeners, | ||
) | ||
initializeFetchListener() | ||
initializeWebSocketListener() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Vadman97 I'm looking at setting up the OTEL provider around here and was curious, why did you have to add this initialization logic multiple times here in addition to at the bottom of this file?
Summary
How did you test this change?
Are there any deployment considerations?
no
Does this work require review from our design team?
no