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] update_ui.go中cleanup方法会删除external-ui目录导致报错 #1651

Open
9 tasks done
lnc1995 opened this issue Nov 12, 2024 · 0 comments
Open
9 tasks done
Labels
bug Something isn't working

Comments

@lnc1995
Copy link

lnc1995 commented Nov 12, 2024

验证步骤

  • 我已经阅读了 文档,了解所有我编写的配置文件项的含义,而不是大量堆砌看似有用的选项或默认值。
  • 我仔细看过 文档 并未解决问题
  • 我已在 Issue Tracker 中寻找过我要提出的问题,并且没有找到
  • 我是中文用户,而非其他语言用户
  • 我已经使用最新的 Alpha 分支版本测试过,问题依旧存在
  • 我提供了可以在本地重现该问题的服务器、客户端配置文件与流程,而不是一个脱敏的复杂客户端配置文件。
  • 我提供了可用于重现我报告的错误的最简配置,而不是依赖远程服务器或者堆砌大量对于复现无用的配置等。
  • 我提供了完整的日志,而不是出于对自身智力的自信而仅提供了部分认为有用的部分。
  • 我直接使用 Mihomo 命令行程序重现了错误,而不是使用其他工具或脚本。

操作系统

Linux

系统版本

Docker

Mihomo 版本

1.18.10

配置文件

external-ui: /etc/mihomo/ui

描述

标题描述的不够准确。实际测试下来有以下的一些问题。
1.external-ui 这个配置目录不存在时,并不会递归建立文件夹,尽管日志输出文件夹不存在,创建中。但是只会创建到mihomo这一步。所以首次运行时,这个目录不存在,一样下载不下来UI。
image

2.代码中更新控制面板时会先下载到/root/.config/mihomo目录,然后解压后移动到external-ui配置的目录。比如我的配置,external-ui: /etc/mihomo/ui,解压前的cleanup方法会删除ui目录,导致err = os.Rename(unzipFolder, u.externalUIPath)这一步代码报错。
这个表现和文档中 “ external-ui-name: xd # 合并为 external-ui/xd 非必须,更新时会更新到指定文件夹,不配置则直接更新到 external-ui 目录 ”的描述并不一致。

3.更新完成后,/root/.config/mihomo目录下解压出来的文件夹没有清理,对Docker用户不太友好,毕竟/root/.config/mihomo这个目录是要映射的,某些系统的文件管理器简直要了命,比如RouterOS,无脑展开所有文件夹,没有层级,没有缩进,这种超多文件的目录简直是灾难。

是不是可以直接下载到external-ui配置的目录解压重命名呢,和mihomo之间分开来

重现方式

external-ui: /etc/mihomo/ui,按照metacubexd项目的示例配置成这样就可以复现

日志

No response

@lnc1995 lnc1995 added the bug Something isn't working label Nov 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant