Skip to content

fix(sdk-crashes): Ignore current stacktrace #75140

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 1 commit into from
Jul 30, 2024

Conversation

philipphofmann
Copy link
Member

Ignore the wrapper function getCurrentStackTrace instead of getSentryException. getCurrentStackTrace is the wrapper function for getting the current stacktrace, and getSentryException uses that function. This was changed with
getsentry/sentry-dart#2072 and shipped in sentry-dart 8.2.1.

Ignore the wrapper function getCurrentStackTrace instead of
getSentryException. getCurrentStackTrace is the wrapper function for
getting the current stacktrace, and getSentryException uses that
function. This was changed with
getsentry/sentry-dart#2072 and shipped in
sentry-dart 8.2.1.
@philipphofmann philipphofmann requested a review from a team July 29, 2024 11:21
@philipphofmann philipphofmann self-assigned this Jul 29, 2024
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 29, 2024
Copy link

codecov bot commented Jul 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.09%. Comparing base (94d9865) to head (26494bc).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #75140      +/-   ##
==========================================
+ Coverage   78.08%   78.09%   +0.01%     
==========================================
  Files        6757     6757              
  Lines      301570   301570              
  Branches    51889    51889              
==========================================
+ Hits       235488   235522      +34     
+ Misses      59732    59706      -26     
+ Partials     6350     6342       -8     
Files Coverage Δ
...ry/utils/sdk_crashes/sdk_crash_detection_config.py 100.00% <ø> (ø)

... and 21 files with indirect coverage changes

@philipphofmann philipphofmann merged commit 3a5b77e into master Jul 30, 2024
51 checks passed
@philipphofmann philipphofmann deleted the fix/sdk-crashes-dart-get-current-stacktrace branch July 30, 2024 06:55
roaga pushed a commit that referenced this pull request Jul 31, 2024
Ignore the wrapper function getCurrentStackTrace instead of
getSentryException. getCurrentStackTrace is the wrapper function for
getting the current stacktrace, and getSentryException uses that
function. This was changed with
getsentry/sentry-dart#2072 and shipped in
sentry-dart 8.2.1.
@github-actions github-actions bot locked and limited conversation to collaborators Aug 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants