Skip to content

Conversation

@Zylphrex
Copy link
Member

No description provided.

@Zylphrex Zylphrex requested a review from a team as a code owner November 25, 2025 18:07
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 25, 2025

attributes = list(attribute_keys.values())
sentry_sdk.set_context("api_response", {"attributes": attributes})
return attributes
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Missing span debug data in sentry conventions branch

The span.set_data() calls for attribute_count and attribute_type are only executed in the use_sentry_conventions=False branch. When use_sentry_conventions is True, the function returns early at line 319 without setting this debug data on the span. Given this PR intends to add debug data to the endpoint, this inconsistency appears to be an oversight.

Additional Locations (1)

Fix in Cursor Fix in Web

@Zylphrex Zylphrex merged commit 74b88cb into master Nov 25, 2025
67 checks passed
@Zylphrex Zylphrex deleted the txiao/chore/add-debug-data-to-attribute-items-endpoint branch November 25, 2025 18:43
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.

3 participants