Skip to content

Conversation

trask
Copy link
Member

@trask trask commented Oct 3, 2025

Lettuce 5.1 appears to be the only instrumentation in the entire repo that sets span status description.

I think this is ok to do in a minor version bump, but I could also hide it behind database stability opt-in.

Noticed this while reviewing #8422, which shows the sensitive data showing up in otel.status_description (in addition to exception.message).

Related to

Specifically motivated by open-telemetry/opentelemetry-specification#3496 (comment)

It may be a bit easier to explain to users that there's just one place (exception.message) where this shows up and that users may need to redact exception.message (which will likely be going to log pipeline in the future by default), as opposed to showing up in two places / signals.

@trask trask marked this pull request as ready for review October 3, 2025 23:05
@trask trask requested a review from a team as a code owner October 3, 2025 23:05
@laurit laurit merged commit ede4a2c into open-telemetry:main Oct 6, 2025
221 of 225 checks passed
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.

2 participants