Skip to content

okhttp spring autoconfig documentation - #19555

Draft
lbloder wants to merge 3 commits into
masterfrom
lbloder/okhttp-autoconfig
Draft

lbloder wants to merge 3 commits into
masterfrom
lbloder/okhttp-autoconfig

Conversation

@lbloder

@lbloder lbloder commented Sep 22, 2026

Copy link
Copy Markdown
Collaborator

DESCRIBE YOUR PR

Update okhttp documentation for spring autoconfiguration added in getsentry/sentry-java#5797

IS YOUR CHANGE URGENT?

Help us prioritize incoming PRs by letting us know when the change needs to go live.
Select exactly one option. For deadlines, replace YYYY-MM-DD with the due date. You can update this information later by editing the PR description.

  • Urgent deadline (GA date, etc.): YYYY-MM-DD
  • Other deadline: YYYY-MM-DD
  • No deadline: Not urgent, can wait up to 1 week+

@vercel

vercel Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
develop-docs Ready Ready Preview Sep 28, 2026 12:54pm UTC
sentry-docs Ready Ready Preview Sep 28, 2026 12:54pm UTC

Request Review

@github-actions github-actions Bot added the Priority: Needs Triage The PR description is missing valid priority information label Sep 22, 2026
@codeowner-assignment
codeowner-assignment Bot requested a review from a team September 22, 2026 12:52

<Alert>

Sentry skips auto-configuration when its <PlatformLink to="/opentelemetry/setup/agentless/">OpenTelemetry agentless integration</PlatformLink> or <PlatformLink to="/opentelemetry/setup/agent/">Java Agent</PlatformLink> is on the classpath because OpenTelemetry already instruments OkHttp.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We need an entry for ignoring auto.http.okhttp in SpanUtils.ignoredSpanOriginsForOpenTelemetry.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

double check whether OpenTelemetry agentless integration actually instruments OkHttp
remove the ConditionalOnMissingBean if not

@adinauer adinauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

This branch was successfully deployed

2 active deployments
Preview – sentry-docs — 2eb40972 Deployed Sep 28, 2026 by vercel[bot]
Preview – develop-docs — 2eb40972 Deployed Sep 28, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Needs Triage The PR description is missing valid priority information

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants