Comfy-Org / Comfy-Org/ComfyUI-Manager

Wrong suggestion on unrelated 403

Open
#2,453 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
16.1k
Forks
2.5k
Avg merge
5d 4h
Merged PRs (30d)
13

Description

I was really stumped, because ComfyUI-Manager kept reporting `This action is not allowed with this security level configuration.` but my security level was already set to `weak` by default (using YanWenKun/ComfyUI-Docker).

It turns out that ComfyUI-Manager will report this on any HTTP 403 for these kind of calls. Even if the 403 is not because of the security level. In my case I spent a long time messing with ComfyUI config files, versions, restarting, etc.

It turned out that ComfyUI itself will return a 403 on _potentially modifying_ HTTP methods such as PATCH, POST, PUT – but not on GET, so things seem to work – when the Host header doesn't match the Origin header. Nothing was wrong with my ComfyUI set-up. I needed to forward the Host header in the reverse proxy that was running between my browser and ComfyUI.

It would be nice if ComfyUI-Manager would not return a false suggestion that it is because of the security level in these cases. However I'm not sure if ComfyUI-Manager _can_ detect such cases.

At least I hope this will document the issue so it's easier to find for others.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the HTTP 403 handling that produces the security-level message, then reproduce a modifying request with mismatched Host and Origin headers. Done means an unrelated 403 no longer suggests changing the security level, while genuine security-level failures retain the appropriate message.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
42/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.