Comfy-Org / Comfy-Org/ComfyUI

ComfyUI Electron App Fails to launch GUI if launch args contain tls-certificate

Open
#8,886 3 comments 2 reactions 0 assignees View on GitHub
bug-cop:response-received Potential Bug
Dominant language
Python
Stars
133k
Forks
15.7k
Avg merge
1d 7h
Merged PRs (30d)
158

Description

### Custom Node Testing

- [x] I have tried disabling custom nodes and the issue persists (see [how to disable custom nodes](https://docs.comfy.org/troubleshooting/custom-node-issues#step-1%3A-test-with-all-custom-nodes-disabled) if you need help)

### Expected Behavior

Normally when you launch the ComfyUI electron App, it runs the startup then launches the GUI in window automatically. It should happen when you see "To see the GUI go to: https://127.0.0.1:8000".

### Actual Behavior

With TLS enabled by setting the server config paths for the TLS key and cert file, the ComfyUI electron app fails to automatically launch the GUI and gets stuck on the log window.

You can still access ComfyUI server through the listed https url by browser.
However, accessing it this way doesn't give you the option to change server config options, meaning there is no way to remove the TLS file paths (without editing the comfy.settings.json file directly)

### Steps to Reproduce

1. Open the ComfyUI electron app
2. Open the Settings gear icon in the bottom left
3. Select Server-Config at the bottom of the settings list
4. Set a valid path to your TLS Key and Cert files
5. Restart ComfyUI

Notably, this was only tested with a self-signed certificate.

### Debug Logs

```powershell
The output logs from the Electron App on startup can't be copied.
But no error is produced and the logs look identical other than using https instead of http.
```

### Other

_No response_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.