Skip to content
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

crashpad needs to offer zip of attachments #51

Open
alecazam opened this issue Oct 26, 2023 · 3 comments
Open

crashpad needs to offer zip of attachments #51

alecazam opened this issue Oct 26, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@alecazam
Copy link

Archiving multiple attachments building an archive , or even compressing a text log file would be valuable. Our log files are all text, and Windows vs. SteamDeck zip compression should probably be inside the crash handler.

@alecazam alecazam changed the title crashpad needs to offer zip ofattachments crashpad needs to offer zip of attachments Oct 26, 2023
@gm4sl gm4sl added the enhancement New feature or request label Oct 30, 2023
@alecazam
Copy link
Author

And also being able to read deflated and archived files in Backtrace would complete the work. At least logs.

@konraddysput
Copy link
Collaborator

Windows vs. SteamDeck zip compression should probably be inside the crash handler.

Can you elaborate more on why? Right now we expect Crashpad to send what it gets. Our integration handles everything related to attachment changes. In our system when we want to see "log" we should see it as a log.

In addition to that you have a use case with zip attachments on Windows and steamDeck. How about, iOS, Mac, Android, how about Fushia, and other OS that also uses Crashpad?

Please keep in mind - we do not want to put more and more features in the crash reporting solution. Every kind of problem generated by the crash handler will cause data loss for the client. I have mixed feelings about this as an issue.

@alecazam
Copy link
Author

We can dip and deflate on our end, but then Backtrace needs to display the unzipped or parts of an archive. My only point was that our text-based .log files are big (up to 1mb), and we'd like to compress them when a crash happens. Having every caller have to pull in zip libraries is redundant but if you want to leave it to the caller, then at least have Backtrace deal with decompress when viewing atttachments.

If I have Sky.log.zip, then I'd prefer to see that unzipped in Backtrace when I click on it without having to download it. Archiving multiple attachments would seem useful and simplify uploading multiple attachments to the server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants