-
Notifications
You must be signed in to change notification settings - Fork 1
[RUM-15104] Integrate dd-trace #95
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
Merged
Merged
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
82c25ab
Integrate dd-trace for HTTP resource tracing and automatic preload in…
cdn34dd c692ba6
Update playground app to add more testing options
cdn34dd 010b631
Send all dd-trace spans to span intake with electron context enrichement
cdn34dd be03846
Remove `tracing` configuration & have it enabled by default
cdn34dd f9fbf0a
Remove SDK's preload registration in favor of dd-trace's approach
cdn34dd 09b829e
Rename `init` entry point to `instrument`
cdn34dd 87e3537
Update e2e apps to use instrument entry point for preload injection
cdn34dd 34ec724
Move dd-trace from devDependencies to dependencies
cdn34dd 81d91f4
Add Vite bundler plugin for dd-trace integration
cdn34dd de4a136
Add Webpack bundler plugin for dd-trace integration
cdn34dd 28a1837
Add build config and package exports for bundler plugins
cdn34dd 5d56960
Update e2e integration apps to use bundler plugins
cdn34dd 6bb0117
Update documentation for bundler plugins
cdn34dd d347711
Remove SDK preload scripts in favor of dd-trace's preload injection
cdn34dd 26da0c8
Fix ResourceConverter: precise SDK filtering and consistent service name
cdn34dd 393ef1e
Add unit tests for ResourceConverter
cdn34dd 2e3262b
Fix ESM compatibility for require('dd-trace') calls
cdn34dd f2ce07a
Add esbuild plugin
cdn34dd 2ae8e60
Switch dd-trace from local tgz to published 5.103.0
cdn34dd 8fa4262
Add externalization and node_modules copy to DatadogWebpackPlugin
cdn34dd a85724b
Merge branch 'main' into carlosnogueira/RUM-15104/integrate-dd-trace
bcaudan 5766b26
Fix ESM preload injection in esbuild plugin
cdn34dd 2ad720f
👌 rework README
bcaudan 6c9beed
👌 rework architecture
bcaudan 7c1effe
Make bundler plugins self-contained: no manual instrument import needed
cdn34dd 03a5a5c
✅ add e2e tests for main-process resource collection
bcaudan 35e20f2
✨ align proxy URL with ddforward pattern
bcaudan d34a414
✅ ensure traces are sent for network calls
bcaudan 9486b5f
✅ ensure traces are sent for ipc calls
bcaudan 437a5f6
✅ add integration scenario for main-process fetch trace
bcaudan 989ba5d
Add ESM output support to vite plugin
cdn34dd bb1f449
Add electron-vite ESM integration test app
cdn34dd 52195f0
Add mainFetch IPC to electron-vite-esm app and increase intake timeout
cdn34dd 2b0073c
✨ add esbuild integration apps for CJS and ESM
bcaudan d0e4bbf
PR review fixes
cdn34dd 2ded3ee
Rename ResourceConverter to SpanProcessor and extract methods
cdn34dd 2065973
👌 filtering span corresponding to the intake
bcaudan 6fb7ace
♻️ use process.getSystemVersion for macOS user agent
bcaudan 7ecfce1
👌 do not consider dd-trace as third party
bcaudan 3249740
👌 avoid type duplication
bcaudan 539d819
👌 Discard span when no session or view active
bcaudan e61fcd4
👌 fix + enrich dd-trace links
bcaudan 4076197
👌 rework typings
bcaudan f4b6afb
🔥 remove dd-trace.tgz
bcaudan 620088f
👌 add todo for dd-trace internal access
bcaudan 2f0d983
🐛 date format issue broke span ingestion
bcaudan 107a457
Update architecture documentation for dd-trace integration
cdn34dd 8b4d100
Patch dd-trace span parenting for HTTP requests inside ipcMain.handle…
cdn34dd File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.