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.

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

file endpoints("v1") to use query parameters #6963

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bigcat88
Copy link
Contributor

This is just the rebased #6376 PR.

Frontend Part: Comfy-Org/ComfyUI_frontend#2182

Changes Made:

  • Added new file-related endpoints (getuserdata, post_userdata, delete_userdata, and move_userdata) to use query parameters instead of path parameters for file paths. This ensures compatibility with proxy-servers by avoiding reliance on URL path encoding.
  • For the move_userdata endpoint, both the source and destination paths are now passed as query parameters (source and dest).

Notes:

  • The Frontend must be updated to reflect these changes, as the file paths have been moved from path parameters to query parameters.
  • Old endpoints were not touched, compatibility with old frontends will not be broken.

Resolves: #5629

Partially fixes problem from #4534

…meters

Signed-off-by: bigcat88 <bigcat88@icloud.com>
@StefanoBalocco
Copy link

StefanoBalocco commented Mar 2, 2025

Any news on this?

I stopped to use comfyui because of saves problem with a nginx reverse proxy, I would love to finally use it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

api/userdata/workflows 405
2 participants