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

[Bug] Use raw dir name to serve static web content #6107

Merged
merged 1 commit into from
Dec 23, 2024

Conversation

huchenlei
Copy link
Collaborator

@huchenlei huchenlei commented Dec 19, 2024

Resolves Comfy-Org/manager-core#9

The urllib.parse.quote(name) was added in

9b1d5a5#diff-791d4d41d3718d15d49180f3aacc8370b8cab07383f0d35b2713651cc0adfe46R511

However, it is causing issue if there are special characters in the custom node directory, such as @. Previously the issue was not surfaced as github repo name cannot contain any special character that urlencode to something else, until the repo@version syntax was introduced by the manager.

Screenshot 2024-12-18 at 4 50 07 PM
Screenshot 2024-12-18 at 4 54 52 PM

Step to reproduce

@huchenlei huchenlei changed the title Use raw dir name to serve static web content [Bug] Use raw dir name to serve static web content Dec 19, 2024
@huchenlei huchenlei added Bug Something is confirmed to not be working properly. Important labels Dec 19, 2024
@robinjhuang
Copy link
Collaborator

Discussed with HCL offline. This should not cause any regressions because github does not allow special characters in their repository name anyways. So this was probably never needed in the first place.

@robinjhuang robinjhuang self-requested a review December 20, 2024 01:35
Copy link
Collaborator

@robinjhuang robinjhuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works for me.

@comfyanonymous comfyanonymous merged commit f18ebbd into master Dec 23, 2024
10 checks passed
@comfyanonymous comfyanonymous deleted the fix_manager_cnr branch December 23, 2024 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is confirmed to not be working properly. Important
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Install extension adds @<version> at in the extension folder
3 participants