Skip to content

Conversation

@huihuihuiz
Copy link
Contributor

This PR adds the custom node plugin "lora_downloader".

GitHub repo:
https://github.com/huihuihuiz/lora_downloader

description:
A custom node for downloading, organizing, and managing LoRA models directly inside ComfyUI.

huihuihuiz and others added 4 commits December 7, 2025 23:08
{
    "author": "huihuihuiz",
    "title": "LoRA Downloader for ComfyUI",
    "id": "lora_downloader",
    "reference": "https://github.com/huihuihuiz/lora_downloader",
    "repo_url": "https://github.com/huihuihuiz/lora_downloader",
    "install_type": "git",
    "description": "A ComfyUI custom node for downloading and managing LoRA models directly within the UI."
}
Removed Lora_Downloader entry from extension-node-map.json
@ltdrdata
Copy link
Member

ltdrdata commented Dec 8, 2025

Please use something like os.path.commonpath to ensure that any paths constructed from external input remain within an isolated directory. As it stands, there is a vulnerability that allows writing or reading files at arbitrary locations via relative paths and similar techniques.

https://github.com/huihuihuiz/lora_downloader/blob/28b95a58aa33235adaed7af5dde3f9b1a72abbc8/lora_downloader.py#L43C50-L43C59

https://github.com/huihuihuiz/lora_downloader/blob/28b95a58aa33235adaed7af5dde3f9b1a72abbc8/lora_downloader.py#L108

https://github.com/huihuihuiz/lora_downloader/blob/28b95a58aa33235adaed7af5dde3f9b1a72abbc8/lora_downloader.py#L128

@huihuihuiz
Copy link
Contributor Author

Thanks for the review.

All user-controlled paths have been secured using os.path.commonpath via a safe_join helper, ensuring all file access remains within the allowed directories.

@ltdrdata ltdrdata merged commit 934c994 into Comfy-Org:main Dec 11, 2025
2 checks passed
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.

2 participants