Skip to content

fix(theme-manager): 修复 IPC 守卫误判与 WorkBuddy 安装位置探测 - #3

Open
hncz2015 wants to merge 1 commit into
comeonzhj:mainfrom
hncz2015:main
Open

fix(theme-manager): 修复 IPC 守卫误判与 WorkBuddy 安装位置探测#3
hncz2015 wants to merge 1 commit into
comeonzhj:mainfrom
hncz2015:main

Conversation

@hncz2015

Copy link
Copy Markdown

修复内容

1. IPC 安全守卫误判(报错:拒绝来自非本地页面的请求)

src/main/main.mjsregisterTrustedHandler 原先用 event.senderFrame?.urlpathToFileURL(rendererDirectory/index.html).href 做精确字符串比较。运行时 senderFrame?.url 取不到值(恒为 ""),永远不等于 trustedUrl,守卫误杀合法本地页面,导致 manager:bootstrap 调用被拒、皮肤列表加载失败。

改为:event.sender === mainWindow.webContents 身份校验为首选,并保留归一化 URL 比较作为兜底(归一化解码 %-编码、统一分隔符、小写、去尾部斜杠)。同时让 rendererDirectory 防御性解析,兼容 asar 内 / 解包外两种布局。

2. WorkBuddy 安装位置探测失败(报错:找不到 WorkBuddy 安装位置)

src/main/runtime.mjswindowsCandidates() 原先只列 C 盘 4 个固定目录,而 WorkBuddy 实际装在非 C 盘(如 G:/WKBD/WorkBuddy/WorkBuddy.exe)时永远找不到。

改为:增加 C~Z 全盘符扫描,每个盘查 WKBD/WorkBuddy/WorkBuddy.exeWorkBuddy/WorkBuddy.exe,兼容任意盘符的自定义 / 便携安装。

验证

  • main.mjs / runtime.mjs 语法检查通过。
  • 本机实测 resolveWorkBuddyPath() 正确解析到 G:\WKBD\WorkBuddy\WorkBuddy.exe
  • 重新打包 app.asar 后运行皮肤管理器不再报上述两处错误,主题列表正常加载。

1. registerTrustedHandler: 将脆弱的 senderFrame.url 精确字符串比较改为
   event.sender === mainWindow.webContents 身份校验,并保留归一化 URL 兜底,
   修复 Windows 下报 '拒绝来自非本地页面的请求' 导致皮肤列表加载失败的问题。
2. windowsCandidates(): 增加 C~Z 全盘符扫描 (WKBD/WorkBuddy/WorkBuddy.exe 与
   WorkBuddy/WorkBuddy.exe),修复 WorkBuddy 装在非 C 盘 (如 G:/WKBD) 时报
   '找不到 WorkBuddy 安装位置' 的问题。
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.

1 participant