[pull] main from getsentry:main - #417
Merged
Merged
Conversation
* fix(core): Clear contexts on Scope.clear() (JAVA-683) Scope.clear() reset every field except contexts, so anything set via setContexts survived a clear. Contexts had no clear() method, unlike the tags, extra, and fingerprint collections, which is why it was skipped. Add Contexts.clear(), override it in CombinedContextsView to delegate to the default context, and clear contexts in Scope.clear() while notifying scope observers so persisted scopes stay in sync. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * changelog --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
* feat(android): Add trace markers to SentryAndroid.init (JAVA-685) Wrap SentryAndroid.init and the user OptionsConfiguration callback in android.os.Trace sections so SDK startup cost shows up as named slices in a Perfetto trace or a CI TraceSectionMetric. Sections use constant names and allocate nothing when no trace is recording. Kept intentionally minimal as the SDK's first shipped Trace instrumentation. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * changelog * feat(android): Keep only the SentryAndroid.init trace section (JAVA-685) Drop the SentryAndroid.init.configure section to keep the SDK's first shipped android.os.Trace instrumentation minimal and easy to review. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> * Remove changelog entry in favor of #skip-changelog * fix(android): Balance init trace section on lock failure (JAVA-685) Move Trace.beginSection above the try-with-resources so it always pairs with the endSection() in the finally. If staticLock.acquire() threw (e.g. OOM allocating the lock), the body was skipped but the finally still ran, popping an unrelated systrace section. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Bumps the github-actions group with 2 updates: [github/codeql-action/init](https://github.com/github/codeql-action) and [github/codeql-action/analyze](https://github.com/github/codeql-action). Updates `github/codeql-action/init` from 4.37.4 to 4.37.5 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@f205ea1...d1ba80a) Updates `github/codeql-action/analyze` from 4.37.4 to 4.37.5 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@f205ea1...d1ba80a) --- updated-dependencies: - dependency-name: github/codeql-action/init dependency-version: 4.37.5 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions - dependency-name: github/codeql-action/analyze dependency-version: 4.37.5 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: github-actions ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…685) (#5903) Read the SentryAndroid.init android.os.Trace section via a TraceSectionMetric so the cold-start macrobenchmark reports SDK-init duration on its own, separate from timeToInitialDisplay. This resolves init-cost changes that the whole-start metric would bury in cold-start noise. Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )