-
-
Notifications
You must be signed in to change notification settings - Fork 306
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
feat: Add screenshot at crash #1920
Merged
Merged
Changes from 23 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
4ba0322
SentryCrash screenshot callback
brustolin 9494db5
add screenshot attachment
brustolin cc44cca
Merge branch 'master' into feat/crash-screenshot
brustolin aca1443
Format code
getsentry-bot a4bad19
Update CHANGELOG.md
brustolin 8c5465c
Merge branch 'feat/crash-screenshot' of https://github.com/getsentry/…
brustolin 66820bc
Apply suggestions from code review
brustolin 23d3536
Improvements with file interactions
brustolin ae6b9e0
Merge branch 'master' into feat/crash-screenshot
brustolin d60054c
string format
brustolin 4eadd21
Format code
getsentry-bot dcfb4f9
Merge branch 'master' into feat/crash-screenshot
brustolin 7f66d8d
Merge branch 'feat/crash-screenshot' of https://github.com/getsentry/…
brustolin a723c7a
Report sink test
brustolin 636bcbe
integration test
brustolin 511864a
Update SentryScreenshot.m
brustolin db0865e
Merge branch 'master' into feat/crash-screenshot
brustolin e2fa1f2
tests
brustolin ceb5fbe
Format code
getsentry-bot 453dddf
Update SentryCrash.m
brustolin cbe448f
Merge branch 'feat/crash-screenshot' of https://github.com/getsentry/…
brustolin 60a7c2e
Update SentryCrashReportStore.c
brustolin 5229655
Format code
getsentry-bot da48b93
Apply suggestions from code review
brustolin dbf5d0b
testfs
brustolin bd84076
Merge branch 'feat/crash-screenshot' of https://github.com/getsentry/…
brustolin 64845d0
Update SentryCrash.m
brustolin 6a66e78
Format code
getsentry-bot 0b5531a
Update SentryCrashReportStore_Tests.m
brustolin 9b880f6
Format code
getsentry-bot 8c66ac8
more fixes
brustolin 068179d
Merge branch 'feat/crash-screenshot' of https://github.com/getsentry/…
brustolin bf6008e
Merge branch 'master' into feat/crash-screenshot
brustolin File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
There are no files selected for viewing
This file contains 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 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 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 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 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 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 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 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 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 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 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 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 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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
h
: When do we delete the screenshots? The attachments will be read when the SDK creates the envelope. How do we know that the screenshots are still stored on disk? Maybe it's better to read the attachments already here, and useSentryAttachment .initWithData
. Then we could delete them already here.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Screenshots are deleted alongside the report. We had a code for that already.
Lets say we fail to transmit the report, on the next attempt the files won't be there, I rather have the current mechanism that deletes it.