Skip to content

feat(data_export): add logging for data-export-failure#112169

Merged
JoshuaKGoldberg merged 1 commit intomasterfrom
data-export-failure-logs
Apr 7, 2026
Merged

feat(data_export): add logging for data-export-failure#112169
JoshuaKGoldberg merged 1 commit intomasterfrom
data-export-failure-logs

Conversation

@JoshuaKGoldberg
Copy link
Copy Markdown
Member

We already have logging for data-export-success. That success log includes the user's email, which makes it nice and easy to tell if someone's data export has succeeded (at least this far in code). But there's no equivalent log for failure. Which is what @narsaynorath and I were trying to debug earlier today and having to do all sorts of log/trace query workarounds for.

Fixes LOGS-673.

@linear-code
Copy link
Copy Markdown

linear-code bot commented Apr 2, 2026

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 2, 2026
@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review April 2, 2026 21:14
@JoshuaKGoldberg JoshuaKGoldberg requested a review from a team as a code owner April 2, 2026 21:14
@JoshuaKGoldberg JoshuaKGoldberg merged commit 4dda701 into master Apr 7, 2026
66 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the data-export-failure-logs branch April 7, 2026 15:24
george-sentry pushed a commit that referenced this pull request Apr 9, 2026
We already have logging for `data-export-success`. That success log
includes the user's email, which makes it nice and easy to tell if
someone's data export has succeeded (at least this far in code). But
there's no equivalent log for failure. Which is what @narsaynorath and I
were trying to debug earlier today and having to do all sorts of
log/trace query workarounds for.

Fixes LOGS-673.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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