给Electron提的Bug修复啦

大家好,我是老纪。

书接上回《抽丝剥茧:Electron与Node.js的奇葩Bug》,没有看过的读者老爷们推荐阅读一下。

我在ElectronGitHub上提的issue,现在已经关闭了。

我们再简单回顾下这个Bug的背景与产生原因:

  1. Node.js自v18.17.0起,为fs.readdir相关API(同步、异步回调与Promise)都新增了一个参数recursive,但事实上,同步和异步回调的API都有Bug,直到v18.19.0才修复。
  2. Electron v27的多个版本内置的Node.js使用v18.17v18.18,直到v29开始使用Node.js的v20,才没有这个Bug
  3. 由于Electron对fs进行了wrap处理,v29起虽然在大部分情况下没有问题,但是asar模式下内置资源路径的递归没有实现,仍会引发Bug,这也正是此次codebytere修复些问题时的标题:fix: support recursive readdir in Asar files#41582

我们看下codebytere的描述信息:

她其实做了两件事:

  1. 修复asar模式下使用recursive参数的Bug
  2. 规避fs.promises.readdir的性能风险。对于fs.promises.readdir的重写,不再是原来的fs.promises.readdir = util.promisify(fs.readdir),而是依赖于fs.promises.readdir的原生方法本身。原因是我上篇文章提到的,fs.readdir使用recursive参数的异步回调处理是同步的,可能会阻塞进程,有性能上的风险。

从她的代码提交上看,她又手写了两个同步异步的递归函数(代码是从Node.js里借鉴的):

不过我更倾向于把她的762行移到756行,这样可以在readdirRecursive中专注处理asar的逻辑,如果Node.js在这里再有什么Bug,可以随着版本的升级而修复,起码只会影响asar的部分。当然,目前这种修改,对于Electron而言,也无伤大雅,真要有了问题,也就是改多改少的区别了。像这么底层的东西,如果不是这样的API变更,通常是很健壮的。

我这个issue下还有两个开发者反馈遇到了同样的事情:

虽然这个问题关闭了,但新的版本还没有发出来。从目前的提交来看,这个修复只是针对Electron v29+,因为它即使合并到v27v28,由于对应的Node.js版本没有升级到v18.19,其同步和异步回调的API仍是有问题的。

相关推荐
liangshanbo121511 小时前
写好 React useEffect 的终极指南
前端·javascript·react.js
哆啦A梦158813 小时前
搜索页面布局
前端·vue.js·node.js
_院长大人_13 小时前
el-table-column show-overflow-tooltip 只能显示纯文本,无法渲染 <p> 标签
前端·javascript·vue.js
Q_Q51100828514 小时前
python+uniapp基于微信小程序的旅游信息系统
spring boot·python·微信小程序·django·flask·uni-app·node.js
哆啦A梦158815 小时前
axios 的二次封装
前端·vue.js·node.js
阿珊和她的猫15 小时前
深入理解与手写发布订阅模式
开发语言·前端·javascript·vue.js·ecmascript·状态模式
Q_Q51100828515 小时前
python基于web的汽车班车车票管理系统/火车票预订系统/高铁预定系统 可在线选座
spring boot·python·django·flask·node.js·汽车·php
yinuo15 小时前
一行 CSS 就能搞定!用 writing-mode 轻松实现文字竖排
前端
snow@li15 小时前
html5:拖放 / demo / 拖放事件(Drag Events)/ DataTransfer 对象方法
前端·html·拖放
浪裡遊17 小时前
Nivo图表库全面指南:配置与用法详解
前端·javascript·react.js·node.js·php