Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

The home page keeps loading #6780

Closed
Bluewind001 opened this issue Feb 11, 2025 · 5 comments
Closed

The home page keeps loading #6780

Bluewind001 opened this issue Feb 11, 2025 · 5 comments
Labels
Potential Bug User is reporting a bug. This should be tested.

Comments

@Bluewind001
Copy link

Expected Behavior

Open the home page of comfyui

Actual Behavior

Always loading

Steps to Reproduce

ComfyUI-0.3.14 ,
Python 3.12.0 | packaged by Anaconda, Inc. | (main, Oct 2 2023, 17:29:18) [GCC 11.2.0] on linux

run command：
CUDA_VISIBLE_DEVICES=2 python main.py --listen 0.0.0.0 --port 8765 --enable-cors-header '*' --verbose DEBUG --log-stdout --disable-all-custom-nodes

A layer of domain forwarding is added in the middle, causing the access home page to load all the time.

Reason： app/user_manager.py#L170 return 404 code .

Solution: $ mkdir user/default/workflows . It is recommended that you create this folder automatically.

Image

Debug Logs

None

Other

No response
@Bluewind001 Bluewind001 added the Potential Bug User is reporting a bug. This should be tested. label Feb 11, 2025
@8251794
Copy link

8251794 commented Feb 11, 2025

Same problem, how to solve it?

@Bluewind001
Copy link
Author

Solution:
$ cd comfyui/
$ mkdir user/default/workflows

@ltdrdata
Copy link
Collaborator

Under normal circumstances, this code should automatically create the required directories. Check if there are any permission issues.

ComfyUI/app/user_manager.py

Line 165 in 1d5d658

path = self.get_request_user_filepath(request, directory)

ComfyUI/app/user_manager.py

Line 94 in 1d5d658

os.makedirs(parent, exist_ok=True)

@Bluewind001
Copy link
Author

@ltdrdata yeap, it can make new folder, such as parent is user/default in L94 , but the path is user/default/workflows in L165, then it will return 404 of "Directory not found" in L170.
The url requested by the front end is /api/userdata?dir=workflows&recurse=true&split=false&full_info=true

ltdrdata added a commit to ltdrdata/ComfyUI that referenced this issue Feb 15, 2025
@ltdrdata
fixed: An issue where the workflows path is not created if it does …
7a97572 
…not exist in the user directory.

The `file` argument of the `get_request_user_filepath` method is always assumed to be a file, leading to checking the parent directory when creating a directory.
However, it is necessary to distinguish whether the target path is a file or a directory and create the path accordingly.

comfyanonymous#6780 (comment)
ltdrdata added a commit to ltdrdata/ComfyUI that referenced this issue Feb 15, 2025
@ltdrdata
fixed: An issue where the workflows path is not created if it does no…
a476e4c 
…t exist in the user directory.

The `file` argument of the `get_request_user_filepath` method is always assumed to be a file, leading to checking the parent directory when creating a directory.
However, it is necessary to distinguish whether the target path is a file or a directory and create the path accordingly.

comfyanonymous#6780
@ltdrdata ltdrdata mentioned this issue Feb 15, 2025
Open
@ltdrdata
Copy link
Collaborator

This PR will patch this issue.
#6816

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Potential Bug User is reporting a bug. This should be tested.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Bluewind001 @ltdrdata @8251794