errors: remove stack data when sending to sentry
Metadata
Current evaluation
Merged PR #2036 removed stack data from Sentry reports, reducing accidental disclosure of local filesystem paths, in commit 96ccad81c.
Resolution: closed resolved
Reason: PR #2036 was merged and landed as commit 96ccad81c, changing Sentry error reporting to omit local stack data.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2036 removed stack data from Sentry reports, reducing accidental disclosure of local filesystem paths, in commit 96ccad81c. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2036 (commit 96ccad81c, +11/-2, 2 files) removes local stack data from Sentry error reports in snapcraft's errors module, avoiding accidental disclosure of filesystem paths. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2036 removed local stack data from Sentry reports via commit 96ccad81c, reducing accidental disclosure of filesystem paths. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2036 removed stack data from error reports sent to Sentry, implemented by sergiusens across two files. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged changes to remove stack data from error reports sent to Sentry. Approved by kyrofa and implemented by sergiusens across two files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Removes stack trace data before sending error reports to Sentry to reduce payload size and improve privacy. Approved by kyrofa after unit tests passed. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1946 errors: add ability to send stack traces to sentry | snapcraft | closed | Sentry traceback reporting in #1946 was superseded by merged PR #1961, which landed as commit 364596087 with confirmation-gated error reports and closed #1918. | |
| #1947 errors: add ability to send stack traces to sentry | snapcraft | closed | PR #1947 was closed unmerged and superseded by #1961, merged as commit 364596087 with confirmation-gated Sentry traceback reporting for #1918. |