Skip to content

更新 jQuery3.7.1 修复静态文件 授信 host 功能#731

Closed
gaoxingzaq wants to merge 3 commits intokekingcn:masterfrom
gaoxingzaq:master
Closed

更新 jQuery3.7.1 修复静态文件 授信 host 功能#731
gaoxingzaq wants to merge 3 commits intokekingcn:masterfrom
gaoxingzaq:master

Conversation

@gaoxingzaq
Copy link
Copy Markdown
Contributor

No description provided.

@klboke
Copy link
Copy Markdown
Contributor

klboke commented Apr 10, 2026

我这边把这个 PR 拉到本地做了一轮验证,先说结论:

  • tgz 作为压缩包类型这部分功能是生效的,我本地实际测下来已经可以进入“压缩包预览”页面。
  • trust.host 这次针对静态 file= 入口(例如 pdfjs / drawio / heic 等)的处理方向也是合理的,相关入口至少没有被直接改挂。

不过从 review 角度看,我这边有几个建议,主要是希望把这次 PR 再收敛一点:

  1. PR 主题和实际改动范围有点不完全匹配
    这次除了 tgz 支持和 trust.host 相关修改,还混进了不少前端静态资源 / 页面模板调整。功能本身没问题,但会让 review 成本和后续维护成本都明显变高。

  2. 看起来有一批疑似误提交的“备份”文件
    diff 里出现了多份带“备份”痕迹的静态文件(路径里表现成乱码文件名)。这类文件建议确认一下是不是本地临时文件误带进来了;如果不是本次修复必须内容,最好去掉。

  3. server/src/main/resources/static/drawio/js/app.min.js 这块变更量非常大,建议单独说明原因
    这部分几乎是整块替换了 vendor 文件。即使它是合理更新,也建议在 PR 描述里补一句:

    • 为什么需要改这块
    • 是版本升级、重新构建产物,还是误覆盖

    这样后续 reviewer 会更容易判断风险。

如果把这些不相关 / 不必要的改动再收一收,我觉得这个 PR 会更清晰,也更容易放心合入。

@gaoxingzaq gaoxingzaq closed this Apr 11, 2026
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