Skip to content

fix(stt): honor Whisper provider proxy - #9776

Closed
casama233 wants to merge 1 commit into
AstrBotDevs:masterfrom
casama233:codex/fix-whisper-stt-proxy
Closed

fix(stt): honor Whisper provider proxy#9776
casama233 wants to merge 1 commit into
AstrBotDevs:masterfrom
casama233:codex/fix-whisper-stt-proxy

Conversation

@casama233

@casama233 casama233 commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • pass the configured provider proxy to the Whisper STT OpenAI client
  • add regression coverage for proxy client construction

Testing

  • python -m pytest -q /tmp/test_whisper_api_source.py

Summary by Sourcery

Route Whisper STT requests through the configured provider proxy.

Bug Fixes:

  • Honor the configured provider proxy when constructing the Whisper STT OpenAI client.

Tests:

  • Add regression coverage verifying the provider proxy is passed to the Whisper HTTP client.

@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. area:provider The bug / feature is about AI Provider, Models, LLM Agent, LLM Agent Runner. labels Aug 22, 2026

@sourcery-ai sourcery-ai Bot left a comment

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.

Hey - I've reviewed your changes and they look great!

Sourcery assessment

Needs a human reviewer. The change routes Whisper audio through the configured proxy, so a faulty or unintended proxy can expose submitted audio or prevent transcription requests from reaching the provider. Reverting stops the new routing behavior, but cannot undo audio that was already sent through the wrong proxy.


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@casama233

Copy link
Copy Markdown
Contributor Author

Superseded by # to comply with the repository branch and PR template conventions.

@casama233 casama233 closed this Aug 22, 2026
@casama233

Copy link
Copy Markdown
Contributor Author

Superseded by #9777 to comply with the repository branch and PR template conventions.

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

Labels

area:provider The bug / feature is about AI Provider, Models, LLM Agent, LLM Agent Runner. size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant