[PR #24596] [CLOSED] fix: honor SSL verification setting for OpenAI STT #131401

Closed
opened 2026-05-21 16:50:31 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/24596
Author: @Genmin
Created: 5/12/2026
Status: Closed

Base: devHead: fix/stt-ssl-verify


📝 Commits (1)

  • 7090d64 fix: honor SSL verification setting for OpenAI STT

📊 Changes

2 files changed (+34 additions, -0 deletions)

View changed files

📝 backend/open_webui/routers/audio.py (+1 -0)
test/backend/routers/test_audio.py (+33 -0)

📄 Description

Pull Request Checklist

  • Linked Issue/Discussion: Closes #24568.
  • Target branch: The pull request targets the dev branch.
  • Description: A concise description of the changes is provided below.
  • Changelog: A changelog entry following Keep a Changelog format is included below.
  • Dependencies: No new or updated dependencies.
  • Self-Review: The code and regression test were reviewed for scope and consistency.
  • Architecture: Reuses the existing global SSL verification setting; no new setting or flow.
  • Git Hygiene: Atomic branch from current dev with one logical change.
  • Title Prefix: Uses the fix: prefix.

Changelog Entry

Description

  • Honor AIOHTTP_CLIENT_SESSION_SSL for OpenAI-compatible speech-to-text transcription requests made from the audio router.

Fixed

  • Added verify=AIOHTTP_CLIENT_SESSION_SSL to the synchronous requests.post call for {STT_OPENAI_API_BASE_URL}/audio/transcriptions, so deployments using self-signed OpenAI-compatible STT endpoints can disable certificate verification consistently with the rest of the backend HTTP clients.
  • Added a focused regression test that ensures the OpenAI STT transcription request continues to pass the configured SSL verification value.

Additional Information

Validated with:

  • python3 -m pytest test/backend/routers/test_audio.py -q
  • python3 -m py_compile backend/open_webui/routers/audio.py test/backend/routers/test_audio.py
  • uvx ruff check test/backend/routers/test_audio.py
  • uvx ruff format --check test/backend/routers/test_audio.py
  • git diff --check

Full-file Ruff on backend/open_webui/routers/audio.py currently reports pre-existing issues unrelated to this change, so this PR keeps the production diff to the single missing verify keyword.

Screenshots or Videos

  • Backend-only fix; no UI screenshots.

Contributor License Agreement

Note

Deleting the CLA section will lead to immediate closure of your PR and it will not be merged in.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/open-webui/pull/24596 **Author:** [@Genmin](https://github.com/Genmin) **Created:** 5/12/2026 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `fix/stt-ssl-verify` --- ### 📝 Commits (1) - [`7090d64`](https://github.com/open-webui/open-webui/commit/7090d64ff612a8f5db9741b0d015a343a351c658) fix: honor SSL verification setting for OpenAI STT ### 📊 Changes **2 files changed** (+34 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `backend/open_webui/routers/audio.py` (+1 -0) ➕ `test/backend/routers/test_audio.py` (+33 -0) </details> ### 📄 Description # Pull Request Checklist - [x] **Linked Issue/Discussion:** Closes #24568. - [x] **Target branch:** The pull request targets the `dev` branch. - [x] **Description:** A concise description of the changes is provided below. - [x] **Changelog:** A changelog entry following Keep a Changelog format is included below. - [x] **Dependencies:** No new or updated dependencies. - [x] **Self-Review:** The code and regression test were reviewed for scope and consistency. - [x] **Architecture:** Reuses the existing global SSL verification setting; no new setting or flow. - [x] **Git Hygiene:** Atomic branch from current `dev` with one logical change. - [x] **Title Prefix:** Uses the `fix:` prefix. # Changelog Entry ### Description - Honor `AIOHTTP_CLIENT_SESSION_SSL` for OpenAI-compatible speech-to-text transcription requests made from the audio router. ### Fixed - Added `verify=AIOHTTP_CLIENT_SESSION_SSL` to the synchronous `requests.post` call for `{STT_OPENAI_API_BASE_URL}/audio/transcriptions`, so deployments using self-signed OpenAI-compatible STT endpoints can disable certificate verification consistently with the rest of the backend HTTP clients. - Added a focused regression test that ensures the OpenAI STT transcription request continues to pass the configured SSL verification value. --- ### Additional Information Validated with: - `python3 -m pytest test/backend/routers/test_audio.py -q` - `python3 -m py_compile backend/open_webui/routers/audio.py test/backend/routers/test_audio.py` - `uvx ruff check test/backend/routers/test_audio.py` - `uvx ruff format --check test/backend/routers/test_audio.py` - `git diff --check` Full-file Ruff on `backend/open_webui/routers/audio.py` currently reports pre-existing issues unrelated to this change, so this PR keeps the production diff to the single missing `verify` keyword. ### Screenshots or Videos - Backend-only fix; no UI screenshots. ### Contributor License Agreement <!-- 🚨 DO NOT DELETE THE TEXT BELOW 🚨 Keep the "Contributor License Agreement" confirmation text intact. Deleting it will trigger the CLA-Bot to INVALIDATE your PR. Your PR will NOT be reviewed or merged until you check the box below confirming that you have read and agree to the terms of the CLA. --> - [ ] By submitting this pull request, I confirm that I have read and fully agree to the [Contributor License Agreement (CLA)](https://github.com/open-webui/open-webui/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT), and I am providing my contributions under its terms. > [!NOTE] > Deleting the CLA section will lead to immediate closure of your PR and it will not be merged in. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-21 16:50:31 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/open-webui#131401