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.
Which problem is this PR solving?
This PR pulls in the code that will eventually become the OpenTelemetry WebSDK. For now we're using all the code from the draft PR that is up so it is subject to change, we're not worried about it changing too much because it is very similar to the NodeSDK which is a released package.
The main new piece of this SDK is the
SpanIdSpanProcessor
,I have chosen to leave this component out for now because it is pretty unstable and undefined and still depends onNevermind, I figured out how to make it browser friendly because there was prior art in upstream OTel for this with trace and span IDs, this is likely what it will look like so I feel comfortable pulling it in for now.Buffer
which is a Node-only API. We can pull this in as it becomes more stable as it will be an additive change.Short description of the changes
session.id
🎉How to verify that this has the expected result
npm run build
cd examples/hello-world-web
npm run build
npm start