feat: Include web-tracing into the default instrumentations #958
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.
Why
Work in process
For easy of use include the web-tracing instrumentation in the default instrumentations.
For existing users we need to ensure that Faro takes up their already configured
Faro detects duplicated instrumentations and ensures that only one is initialized.
But it uses the first instrument it finds, so it would always pick up the default instrument.
-> Ensure that the user configured instrument takes precedence.
This how the config changes:**
new way with web-tracing auto instrumentation if no customization is needed
New way, with web-tracing auto instrumentation and additional tracing configuration
Current state
What
Links
Checklist