更新 JQ为3.7.1 、添加静态授信htost、 美化报错页面、 新增 tgz 格式支持#732
更新 JQ为3.7.1 、添加静态授信htost、 美化报错页面、 新增 tgz 格式支持#732gaoxingzaq wants to merge 9 commits intokekingcn:masterfrom
Conversation
There was a problem hiding this comment.
修改接入方法 支持file 参数
There was a problem hiding this comment.
取消 index.html 集成到内部 防止XSS
|
Code review summary 整体看下来，这个 PR 我建议先不要合并，当前至少有两个阻断问题：
另外还有一个建议关注的点：
我本地额外做了这些检查：
结论：建议修复上面两个功能性问题后再合并。 |
| const kkagent = '${kkagent}'; | ||
| const baseUrl = '${baseUrl}'.endsWith('/') ? '${baseUrl}' : '${baseUrl}' + '/'; | ||
| if (kkagent === 'true' || !MODEL_URL.startsWith(baseUrl)) { | ||
| MODEL_URL = baseUrl + 'getCorsFile?urlPath=' + encodeURIComponent(Base64.encode(MODEL_URL))+ "&key=${kkkey}"; |
There was a problem hiding this comment.
这里有一个明确的运行时问题：MODEL_URL 在上一行被声明为 const，但这里又重新赋值。只要命中这个代理分支，浏览器就会直接抛出 “Assignment to constant variable”，后面的 3D 预览初始化也会中断。建议改成 let MODEL_URL，或者引入新的 finalModelUrl 变量承接代理后的值。
| // ========== 整合后端变量，直接指定 OFD 文件地址 ========== | ||
| // 后端传入的原始 URL（可能未经过代理） | ||
| var rawUrl = '${finalUrl}'; | ||
| var kkagent = '${kkagent}'; |
There was a problem hiding this comment.
这里更新错变量了：上面真正参与后续 loadOfdFile(...) 调用的是 finalOfdUrl，但这个分支里修改的是 url。结果是 finalOfdUrl 仍然保持原始地址，跨域/代理场景不会生效。建议这里改为 finalOfdUrl = baseUrl + 'getCorsFile?...'，并顺手删掉后面的调试日志。
|
好的 已经修复了 |
2 participants
No description provided.