search

11cafe / comfyui-workspace-manager

A ComfyUI workflows and models management extension to organize and manage all your workflows, models in one place. Seamlessly switch between workflows, as well as import, export workflows, reuse subworkflows, install models, browse your models in a single workspace
MIT License
1.1k stars 52 forks source link

Linux Debain 上 从URL下载模型文件名后携带`";`特殊符号 #339

Closed Llugaes closed 7 months ago

Llugaes commented 7 months ago

环境：Debian GNU/Linux 11

模型链接：https://huggingface.co/comfyanonymous/ControlNet-v1-1_fp16_safetensors/resolve/main/control_lora_rank128_v11f1e_sd15_tile_fp16.safetensors

错误示例：

filename_error

解决方法：

force_filename

成功示例：

filename_correct

大佬能看一下具体是什么问题吗？或者加个开关可以让我toggle这个force_filename？ 像这种问题很多通过链接下载的模型都有，而且只有Linux上出现了，Windows上没有这个问题，目前我暂时写死为True就能解决这个问题，但是很明显是不可持续的，我没有太多时间阅览源码，所以不好贸然提pr，还请大佬定位一下是什么问题，并提供一个官方的解决方案😿plz

Weixuanf commented 7 months ago

hi thanks for reporting, 刚改了一下 应该修好了 你试试吧～感谢提供了code pointer，加快了我debug哈哈 是因为之前我们是从Content-Disposition这个header里提取的filename，之前这个提取的regex match没有写好，有时候会出现一些多余的字符

Llugaes commented 7 months ago

赞👍 测试了一下没问题了， 感谢！