-
-
Notifications
You must be signed in to change notification settings - Fork 386
feat: Standalone app start tracing #7660
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
Open
philipphofmann
wants to merge
52
commits into
main
Choose a base branch
from
feat/standalone-app-start-tracing
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
a43e535
ref: Extract getAppStartMeasurement into SentryAppStartMeasurementProβ¦
philipphofmann 2d02876
ref: Remove resetAppStartMeasurementRead from SentryTracer
philipphofmann ba85ba9
ref: Clean up app start measurement provider API
philipphofmann ec54453
fix: Remove leftover profilerReferenceID in concurrency test
philipphofmann 4d7bfa8
fix: Fix compilation errors in app start measurement provider tests
philipphofmann 87017a5
feat: Add enableStandaloneAppStartTracing option
philipphofmann b31cbf5
feat: Implement standalone app start transaction
philipphofmann 2cad1fb
ref: Pass standalone flag to sentryBuildAppStartSpans
philipphofmann 244ad48
ref: Extract isStandaloneAppStartTransaction method
philipphofmann 6427ad5
ref: Check origin in isStandaloneAppStartTransaction
philipphofmann 683fc13
ref: Move standalone app start check to Swift helper
philipphofmann 81c03f0
ref: Use SentrySpan protocol in app start spans
philipphofmann 51dc82b
fix linter
philipphofmann d4918cb
test: Add standalone app start span building tests
philipphofmann db3ef7e
feat: Guard standalone app start transaction on SDK enabled
philipphofmann 8211d93
ref: Pass app start measurement via tracer configuration
philipphofmann 89686ca
test: Assert span names and operations in standalone test
philipphofmann 985fdd0
test: Improve standalone app start global measurement test
philipphofmann 78e9ab3
fix: Mark app start measurement as read in standalone path
philipphofmann df35dee
feat: Add changelog entry for standalone app start tracing
philipphofmann e0b4ea4
ref: Clarify markAsRead comment as safeguard
philipphofmann 8106a64
fix: Remove trailing commas unsupported by CI Swift version
philipphofmann 661d309
ref: Move SDK enabled guard to top of handle method
philipphofmann 462b028
ref: Log error for unknown app start type
philipphofmann 0c44cc8
ref: Reference SentryAppStartMeasurementProvider in comment
philipphofmann 868d726
ref: Log error for unknown app start type in span builder
philipphofmann a0c07a6
ref: Simplify unknown app start type log message
philipphofmann fc55e12
ref: Simplify unknown app start type log in Swift
philipphofmann a1e88d1
ref: Extract AppStartMeasurementHandler into own file
philipphofmann 529e78a
test: Add tests for AppStartMeasurementHandler
philipphofmann ed652f5
ref: Split sentryBuildAppStartSpans into two methods
philipphofmann 2799506
test: Add integration tests for standalone app start
philipphofmann 6a7e677
test: Add missing import for _SentryPrivate
philipphofmann 8ba3510
test: Replace clearTestState with targeted cleanup
philipphofmann 5106df6
test: Add span validation for standalone app start
philipphofmann 58d04f0
fix: Fix flaky tests by restoring clearTestState and using serializedβ¦
philipphofmann d11bf49
feat: Track standalone app start tracing in enabled features
philipphofmann c7d58ca
ref: address PR review comments
philipphofmann 15b9564
fix: Add visionOS to platform guards in clearTestState
philipphofmann 027eef2
build: update public API after adding experimental option
philipphofmann 4740d03
Merge branch 'main' into feat/standalone-app-start-tracing
philipphofmann 98fbbed
changelog
philipphofmann 8b3d6f2
ref: Remove forward declarations in SentryBuildAppStartSpans
philipphofmann 4819275
ref: Move isStandaloneAppStartTransaction to UIKit section
philipphofmann d0d815e
ref: Rename AppStartMeasurementHandler to AppStartReportingStrategy
philipphofmann f525749
fix: Address review feedback for standalone app start tracing
philipphofmann 9d19a2a
test: Use targeted cleanup instead of blanket clearTestState
philipphofmann 973cc2f
test: Fix test conventions from develop-docs review
philipphofmann 3ac8825
test: Add integration test for standalone app start tracing
philipphofmann ade227c
test: Add coverage for standalone app start edge cases
philipphofmann a243fa7
fix: Remove standalone tracing from sample, fix endif comment
philipphofmann 00e0d76
Merge branch 'main' into feat/standalone-app-start-tracing
philipphofmann 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
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
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.
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.