Skip to content

fix: stack-trace in_app false being omitted#647

Merged
tonyo merged 2 commits intogetsentry:masterfrom
vaind:fix/in-app-false
Jun 5, 2023
Merged

fix: stack-trace in_app false being omitted#647
tonyo merged 2 commits intogetsentry:masterfrom
vaind:fix/in-app-false

Conversation

@vaind
Copy link
Copy Markdown
Contributor

@vaind vaind commented Jun 2, 2023

The stack trace in-app field had omitempty so it wouldn't be sent in JSON when it was false. This wasn't correct as that would actually lead to exactly the opposite behaviour in sentry.io where the frame would show as in-app=true (the default when the field is not present on the frame).

@vaind vaind mentioned this pull request Jun 2, 2023
7 tasks
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 5, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (02e712a) 79.64% compared to head (c77c625) 79.64%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #647   +/-   ##
=======================================
  Coverage   79.64%   79.64%           
=======================================
  Files          38       38           
  Lines        3896     3896           
=======================================
  Hits         3103     3103           
  Misses        683      683           
  Partials      110      110           
Impacted Files Coverage Δ
stacktrace.go 83.65% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@tonyo tonyo merged commit 1b0b536 into getsentry:master Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants