FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat(stdlib): Handle proxy tunnels in httlib integration by sl0thentr0py · Pull Request #5303 · getsentry/sentry-python · GitHub

/ sentry-python Public

feat(stdlib): Handle proxy tunnels in httlib integration - #5303

Merged
sl0thentr0py merged 2 commits into
masterfrom
neel/stdlib-proxy
Jan 19, 2026
ConversationCommits (2)ChecksFiles changed
Merged

feat(stdlib): Handle proxy tunnels in httlib integration#5303
sl0thentr0py merged 2 commits into
masterfrom
neel/stdlib-proxy

Conversation

sl0thentr0py commented Jan 13, 2026

Copy link
Copy Markdown
Member

sl0thentr0py requested a review from a team as a code owner January 13, 2026 15:02

linear Bot commented Jan 13, 2026

Copy link
Copy Markdown

PY-2018 Wrong host names displayed in "Outbound API Requests" when requests are sent via a proxy.

sl0thentr0py changed the title Handle proxy tunnels in httlib integration feat(stdlib): Handle proxy tunnels in httlib integration Jan 13, 2026

github-actions Bot commented Jan 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

New Features ✨

Ai

  • feat(ai): add cache writes for gen_ai by shellmayr in #5319
  • feat(ai): add parse_data_uri function to parse a data URI by constantinius in #5311

Other

  • feat(asyncio): Add on-demand way to enable AsyncioIntegration by sentrivana in #5288
  • feat(openai-agents): Inject propagation headers for HostedMCPTool by alexander-alderman-webb in #5297
  • feat(stdlib): Handle proxy tunnels in httlib integration by sl0thentr0py in #5303
  • feat: Support array types for logs and metrics attributes by alexander-alderman-webb in #5314

Bug Fixes 🐛

Integrations

  • fix(integrations): google genai report image inputs by constantinius in #5337
  • fix(integrations): google-genai: reworked gen_ai.request.messages extraction from parameters by constantinius in #5275
  • fix(integrations): pydantic-ai: properly format binary input message parts to be conformant with the gen_ai.request.messages structure by constantinius in #5251
  • fix(integrations): Anthropic: add content transformation for images and documents by constantinius in #5276
  • fix(integrations): langchain add multimodal content transformation functions for images, audio, and files by constantinius in #5278

Litellm

  • fix(litellm): fix gen_ai.request.messages to be as expected by constantinius in #5255
  • fix(litellm): Guard against module shadowing by alexander-alderman-webb in #5249

Other

  • fix(ai): redact message parts content of type blob by constantinius in #5243
  • fix(clickhouse): Guard against module shadowing by alexander-alderman-webb in #5250
  • fix(gql): Revert signature change of patched gql.Client.execute by alexander-alderman-webb in #5289
  • fix(grpc): Derive interception state from channel fields by alexander-alderman-webb in #5302
  • fix(pure-eval): Guard against module shadowing by alexander-alderman-webb in #5252
  • fix(ray): Guard against module shadowing by alexander-alderman-webb in #5254
  • fix(threading): Handle channels shadowing by sentrivana in #5299
  • fix(typer): Guard against module shadowing by alexander-alderman-webb in #5253
  • fix: Stop suppressing exception chains in AI integrations by alexander-alderman-webb in #5309
  • fix: Send client reports for span recorder overflow by sentrivana in #5310

Documentation 📚

  • docs(metrics): Remove experimental notice by alexander-alderman-webb in #5304
  • docs: Update Python versions banner in README by sentrivana in #5287

Internal Changes 🔧

Release

  • ci(release): Bump Craft version to fix issues by BYK in #5305
  • ci(release): Switch from action-prepare-release to Craft by BYK in #5290

Other

  • chore(gen_ai): add auto-enablement for google genai by shellmayr in #5295
  • ci: 🤖 Update test matrix with new releases (01/19) by github-actions in #5330
  • ci: Add periodic AI integration tests by alexander-alderman-webb in #5313
  • chore: Use pull_request_target for changelog preview by BYK in #5323
  • chore: add unlabeled trigger to changelog-preview by BYK in #5315
  • chore: Add type for metric units by sentrivana in #5312
  • ci: Update tox and handle generic classifiers by sentrivana in #5306

🤖 This preview updates automatically when you update the PR.

sl0thentr0py commented Jan 13, 2026
View reviewed changes
Comment thread sentry_sdk/integrations/stdlib.py Outdated
sentry Bot reviewed Jan 13, 2026
View reviewed changes
Comment thread sentry_sdk/integrations/stdlib.py Outdated
cursor Bot reviewed Jan 13, 2026
View reviewed changes
Comment thread sentry_sdk/integrations/stdlib.py
alexander-alderman-webb approved these changes Jan 13, 2026
View reviewed changes
Comment thread sentry_sdk/integrations/stdlib.py Outdated
tunnel_port = getattr(self, "_tunnel_port", None)
if tunnel_port:
port = tunnel_port
# need to override default_port for correct url recording

alexander-alderman-webb Jan 13, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I'd suggest changing the url recording logic instead.

Otherwise, looks good to me!

sl0thentr0py force-pushed the neel/stdlib-proxy branch 2 times, most recently from 79535e6 to ea832e5 Compare January 19, 2026 13:28
cursor Bot reviewed Jan 19, 2026
View reviewed changes

cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread sentry_sdk/integrations/stdlib.py Outdated
sl0thentr0py force-pushed the neel/stdlib-proxy branch 2 times, most recently from ae1326b to d8e6104 Compare January 19, 2026 13:38
cursor Bot reviewed Jan 19, 2026
View reviewed changes

cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread sentry_sdk/integrations/stdlib.py
sl0thentr0py merged commit 2585678 into master Jan 19, 2026
163 of 165 checks passed
sl0thentr0py deleted the neel/stdlib-proxy branch January 19, 2026 14:14
sentry-release-bot Bot mentioned this pull request Jan 20, 2026
Closed
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

sentry[bot] sentry[bot] left review comments
cursor[bot] cursor[bot] left review comments
alexander-alderman-webb alexander-alderman-webb approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Wrong host names displayed in "Outbound API Requests" when requests are sent via a proxy.

2 participants

Back | FazBrowse Home | New Git URL