Skip to content

feat(ktor): Add URL data to client spans - #6433

Draft
adinauer wants to merge 2 commits into
9.x.xfrom
fix/ktor-client-url-span-data
Draft

feat(ktor): Add URL data to client spans#6433
adinauer wants to merge 2 commits into
9.x.xfrom
fix/ktor-client-url-span-data

Conversation

@adinauer

@adinauer adinauer commented Oct 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

📜 Description

Apply resolver-filtered URL query parameters and fragments to Ktor HTTP client spans as http.query and http.fragment data. Span descriptions continue to exclude query parameters and fragments.

💡 Motivation and Context

Ktor client spans currently omit URL detail data that other HTTP client integrations provide. This adds the data through the shared UrlUtils path so URL query deny lists and off mode are respected.

The behavior was intentionally deferred from #5983 to the 9.0 major release because legacy, unconfigured data-collection settings preserve query values.

resolves: #5984
refs: JAVA-701

💚 How did you test it?

  • ./gradlew :sentry-ktor-client:check
  • ./gradlew spotlessApply apiDump

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

None.

@adinauer @claude
feat(ktor): Add URL data to client spans
a03d1f8 
Apply resolver-filtered query parameters and fragments to Ktor client
spans while keeping span descriptions free of URL details. Respect URL
query collection deny lists and off mode.

Fixes #5984
Co-Authored-By: Claude <noreply@anthropic.com>
@linear-code

linear-code Bot commented Oct 30, 2026

Copy link
Copy Markdown

JAVA-701

@github-actions

github-actions Bot commented Oct 30, 2026

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against beb93a5

@sentry

sentry Bot commented Oct 30, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.62.0 (1) release

⚙️ sentry-android Build Distribution Settings

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@romtsn romtsn Awaiting requested review from romtsn romtsn will be requested when the pull request is marked ready for review romtsn is a code owner
@markushi markushi Awaiting requested review from markushi markushi will be requested when the pull request is marked ready for review markushi is a code owner
@runningcode runningcode Awaiting requested review from runningcode runningcode will be requested when the pull request is marked ready for review runningcode is a code owner
@0xadam-brown 0xadam-brown Awaiting requested review from 0xadam-brown 0xadam-brown will be requested when the pull request is marked ready for review 0xadam-brown is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@adinauer