[PR #1647] [MERGED] fix: Websocket Connection failed with ComfyUI server over HTTPS #7536

New Issue

GiteaMirror · 2025-11-11T17:29:26-06:00

GiteaMirror commented

2025-11-11 17:29:26 -06:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/1647
Author: @dyamagishi
Created: 4/21/2024
Status: ✅ Merged
Merged: 4/21/2024
Merged by: @tjbck

Base: dev ← Head: comfyui_ws_schema

📝 Commits (1)

489c45f fix: Update websocket protocol based on the original schema.

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 backend/apps/images/utils/comfyui.py (+2 -2)

📄 Description

Pull Request Checklist

Description: Briefly describe the changes in this pull request.
Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
Documentation: Have you updated relevant documentation?
Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?

Description

This PR fixes an issue where the websocket fails to connect when the ComfyUI server is secured with HTTPS
(through a reverse proxy or similar).

Changelog Entry

Added

N/A

Fixed

Updated the WebSocket protocol handling to use ws:// for HTTP and wss:// for HTTPS in the comfyui_generate_image function to correct WebSocket connections.

Changed

N/A

Removed

N/A

_{🔄 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/1647 **Author:** [@dyamagishi](https://github.com/dyamagishi) **Created:** 4/21/2024 **Status:** ✅ Merged **Merged:** 4/21/2024 **Merged by:** [@tjbck](https://github.com/tjbck) **Base:** `dev` ← **Head:** `comfyui_ws_schema` --- ### 📝 Commits (1) - [`489c45f`](https://github.com/open-webui/open-webui/commit/489c45ffdf33ef9c9b97e3c52a016d6bf40b09cb) fix: Update websocket protocol based on the original schema. ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `backend/apps/images/utils/comfyui.py` (+2 -2) </details> ### 📄 Description ## Pull Request Checklist - [x] **Description:** Briefly describe the changes in this pull request. - [x] **Changelog:** Ensure a changelog entry following the format of [Keep a Changelog](https://keepachangelog.com/) is added at the bottom of the PR description. - [ ] **Documentation:** Have you updated relevant documentation? - [ ] **Dependencies:** Are there any new dependencies? Have you updated the dependency versions in the documentation? --- ## Description This PR fixes an issue where the websocket fails to connect when the ComfyUI server is secured with HTTPS (through a reverse proxy or similar). --- ### Changelog Entry ### Added - N/A ### Fixed - Updated the WebSocket protocol handling to use ws:// for HTTP and wss:// for HTTPS in the comfyui_generate_image function to correct WebSocket connections. ### Changed - N/A ### Removed - N/A --- <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 2025-11-11 17:29:26 -06:00

GiteaMirror closed this issue

2025-11-11 17:29:26 -06:00

GiteaMirror referenced this issue

2025-11-11 18:08:26 -06:00

[PR #7536] [MERGED] style: Add proper favicon and iOS home screen support #8875

Sign in to join this conversation.