Skip to content

Conversation

JoshuaMoelans
Copy link
Member

@JoshuaMoelans JoshuaMoelans commented Aug 29, 2025

Fixes #1342


Truncation occurrences (‼️ List is WIP):

🔍 sentry_unwind_stack(..., max_frames)

Code here. Need to investigate if this is something we should be truncating still, or if Relay can handle this.

🔍 inproc MAX_FRAMES

Code here and used for unwinding the stack here

Keep truncation

set_tag_n

We keep SDK-side truncation as span tags are soon to be deprecated (replaced by span attributes, which have way larger length limits (or possibly have no limited)).

code here and mention of truncation for sentry_span_set_tag here

removed in (commit) but seems like this is not truncated during ingestion... docs mention that tags must be less than 200 characters
Screenshot 2025-08-29 at 10 47 40

Product doesn't seem to like these large values though 😅
Screenshot 2025-08-29 at 10 53 03

Copy link

github-actions bot commented Aug 29, 2025

Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- SDK Truncation Logic ([#1347](https://github.com/getsentry/sentry-native/pull/1347))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description or adding a skip-changelog label.

Generated by 🚫 dangerJS against a9712b2

@JoshuaMoelans JoshuaMoelans changed the title no longer truncate tag value chore: SDK Truncation Logic Sep 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore: SDK Truncation Logic
1 participant