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>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 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)
489c45ffix: 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
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
Fixed
Changed
Removed
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.