feat: Add AddToTransactions to SentryAttachment by jamescrosswell · Pull Request #5182 · getsentry/sentry-dotnet

jamescrosswell · 2026-05-04T04:11:57Z

Summary

Adds AddToTransactions boolean property to SentryAttachment (defaults to false)
New constructor overload: SentryAttachment(type, content, fileName, contentType, addToTransactions)
Envelope.FromTransaction now accepts optional attachments parameter and includes those with AddToTransactions = true
SentryClient.CaptureTransaction filters attachments to only pass AddToTransactions = true ones to the envelope

Implements #750. See also the Java reference implementation and the SDK spec.

Note

We didn't add this for Maui Screenshots since those get added automatically/internally by the Sentry SDK and only for Events and Feedback (never for transactions - which makes sense - we don't want to capture a screenshot every time a transaction is finished).

Docs

feat(dotnet): document sending attachments with transactions sentry-docs#18741

Changelog Entry

Attachments can now be sent with transactions by setting AddToTransactions on SentryAttachment #5182

Adds an `AddToTransactions` boolean property to `SentryAttachment` (default `false`) that controls whether attachments are included in transaction envelopes. Previously attachments were never sent with transactions; now those with `AddToTransactions = true` are included. Closes #750 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

codecov · 2026-05-04T04:27:36Z

Codecov Report

❌ Patch coverage is 95.65217% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 74.54%. Comparing base (7b09863) to head (2496a53).
⚠️ Report is 15 commits behind head on main.

Files with missing lines	Patch %	Lines
src/Sentry.Maui/Internal/ScreenshotAttachment.cs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5182      +/-   ##
==========================================
+ Coverage   74.23%   74.54%   +0.31%     
==========================================
  Files         509      512       +3     
  Lines       18412    18652     +240     
  Branches     3604     3655      +51     
==========================================
+ Hits        13668    13904     +236     
- Misses       3870     3874       +4     
  Partials      874      874

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Replace the optional-parameter signature with two explicit overloads: - original single-arg overload delegates to the new one (binary compat) - new overload takes logger + attachments without defaults Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The two focused tests (true includes, false excludes) already cover the filtering logic completely. The mixed-attachments test added no additional coverage. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Adds an overloaded ViewHierarchyAttachment constructor accepting addToTransactions and forwards it to the base type, and plumbs the flag through the internal ScreenshotAttachment constructor. Addresses the PR review question about offering the flag consistently on derived attachment types. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

cursor

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

^{Reviewed by Cursor Bugbot for commit a8edc5d. Configure here.}

Adds a test asserting that a null element in the attachments collection is skipped (not added as an envelope item) and that a warning is logged. Addresses the PR review coverage nitpick. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Adds an optional addToTransactions parameter to the test helper and uses it to shorten the AddToTransactionsTrue test, keeping it consistent with the AddToTransactionsFalse test. Addresses the PR review suggestion. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

…t-add-to-transactions # Conflicts: # test/Sentry.Tests/SentryClientTests.cs

hint.Attachments is a mutable public collection, so it can contain a null entry. Filtering it with Where(a => a.AddToTransactions) dereferenced each element and threw a NullReferenceException, dropping the whole transaction before Envelope.FromTransaction could skip the null. The event and feedback paths already defer null handling to the envelope's IsNull guard; mirror that by letting null entries flow through the client-side filter instead of dereferencing them. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

dingsdax

I guess this should get some changelog/docs updates too, implementation looks good, one comment about hint being possibly null

> ℹ️ **Note** > > This PR body was truncated due to platform limits. This PR contains the following updates: | Package | Change | [Age](https://docs.renovatebot.com/merge-confidence/) | [Confidence](https://docs.renovatebot.com/merge-confidence/) | |---|---|---|---| | [Sentry.Maui](https://sentry.io/) ([source](https://github.com/getsentry/sentry-dotnet)) | `6.7.0` → `6.8.0` | ![age](https://developer.mend.io/api/mc/badges/age/nuget/Sentry.Maui/6.8.0?slim=true) | ![confidence](https://developer.mend.io/api/mc/badges/confidence/nuget/Sentry.Maui/6.7.0/6.8.0?slim=true) | --- ### Release Notes <details> <summary>getsentry/sentry-dotnet (Sentry.Maui)</summary> ### [`v6.8.0`](https://github.com/getsentry/sentry-dotnet/blob/HEAD/CHANGELOG.md#680) [Compare Source](getsentry/sentry-dotnet@6.7.0...6.8.0) ##### Features ✨ ##### Logs - feat(logs): add `log4net` integration by [@Flash0ver](https://github.com/Flash0ver) in [#5172](getsentry/sentry-dotnet#5172) - feat(logs): add `NLog` integration by [@Flash0ver](https://github.com/Flash0ver) in [#5176](getsentry/sentry-dotnet#5176) ##### Other - feat(serilog): support restrictedToMinimumLevel when configuring Serilog in code by [@jamescrosswell](https://github.com/jamescrosswell) in [#5181](getsentry/sentry-dotnet#5181) - Attachments can now be sent with transactions by setting `AddToTransactions` on `SentryAttachment` [#5182](getsentry/sentry-dotnet#5182) by [@jamescrosswell](https://github.com/jamescrosswell) in [#5182](getsentry/sentry-dotnet#5182) - Added `SentrySdk.RecordTransaction` to record already-completed transactions and spans (e.g. replayed through a proxy) [#5333](getsentry/sentry-dotnet#5333) by [@jamescrosswell](https://github.com/jamescrosswell) in [#5333](getsentry/sentry-dotnet#5333) - The `Environment` set on the `Scope` now gets synchronized to the native layers (`sentry-cocoa` and `sentry-native`) by [@bitsandfoxes](https://github.com/bitsandfoxes) in [#5365](getsentry/sentry-dotnet#5365) ##### Fixes 🐛 - fix: `SentrySpanProcessor` no longer leaks spans whose Activity never ends (e.g. aborted requests); the Activity is now held via a `WeakReference` so orphaned spans are pruned once it is garbage-collected. by [@Ermabo](https://github.com/Ermabo) in [#5393](getsentry/sentry-dotnet#5393) - The SDK was incorrectly ignoring server rate limits for errors, check-ins, and logs by [@jamescrosswell](https://github.com/jamescrosswell) in [#5412](getsentry/sentry-dotnet#5412) - fix: BackpressureMonitor.Dispose() no longer deadlocks on single-threaded targets by [@jamescrosswell](https://git...

jamescrosswell requested a review from Flash0ver as a code owner May 4, 2026 04:11

jamescrosswell marked this pull request as draft May 4, 2026 05:25

Windows verify tests

39fe419

sentry-warden Bot reviewed May 5, 2026

View reviewed changes

Comment thread src/Sentry/Protocol/Envelopes/Envelope.cs

jamescrosswell and others added 3 commits May 5, 2026 16:29

Windows verify tests

29b2db3

test: Remove redundant mixed-attachments transaction test

0ee50c0

The two focused tests (true includes, false excludes) already cover the filtering logic completely. The mixed-attachments test added no additional coverage. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

jamescrosswell marked this pull request as ready for review May 7, 2026 04:54

jamescrosswell linked an issue May 18, 2026 that may be closed by this pull request

Add addToTransactions to Attachment #750

Closed

Flash0ver reviewed Jul 10, 2026

View reviewed changes

This was referenced Jul 13, 2026

Swap attachments/logger parameter order in Envelope.From* factory methods #5379

Open

Expose AddToTransactions flag on derived SentryAttachment types (ViewHierarchyAttachment, ScreenshotAttachment) #5380

Closed

github-actions Bot added the risk: medium PR risk score: medium label Jul 13, 2026

cursor Bot reviewed Jul 13, 2026

View reviewed changes

Comment thread src/Sentry/SentryClient.cs Outdated

jamescrosswell and others added 2 commits July 13, 2026 14:28

sentry Bot reviewed Jul 13, 2026

View reviewed changes

Comment thread src/Sentry/SentryClient.cs Outdated

jamescrosswell and others added 2 commits July 13, 2026 14:37

Merge remote-tracking branch 'origin/main' into feature/add-attachmen…

895c64e

…t-add-to-transactions # Conflicts: # test/Sentry.Tests/SentryClientTests.cs

jamescrosswell requested a review from Flash0ver July 13, 2026 05:01

dingsdax requested review from bitsandfoxes and dingsdax and removed request for Flash0ver July 15, 2026 23:17

dingsdax reviewed Jul 15, 2026

View reviewed changes

Comment thread src/Sentry/SentryClient.cs

dingsdax self-requested a review July 15, 2026 23:35

dingsdax reviewed Jul 15, 2026

View reviewed changes

jamescrosswell mentioned this pull request Jul 16, 2026

feat(dotnet): document sending attachments with transactions getsentry/sentry-docs#18741

Merged

6 tasks

github-actions Bot mentioned this pull request Jul 27, 2026

chore(deps): update .NET SDK to v6.8.0 lucas-zimerman/sentry-unity#365

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

feat: Add AddToTransactions to SentryAttachment - #5182

feat: Add AddToTransactions to SentryAttachment#5182
jamescrosswell merged 12 commits into
mainfrom
feature/add-attachment-add-to-transactions

jamescrosswell commented May 4, 2026 •

edited

Loading

Uh oh!

codecov Bot commented May 4, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dingsdax left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Uh oh!

Conversation

jamescrosswell commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Docs

Changelog Entry

Uh oh!

codecov Bot commented May 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dingsdax left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

jamescrosswell commented May 4, 2026 •

edited

Loading

codecov Bot commented May 4, 2026 •

edited

Loading