log4js-node / log4js-node/streamroller

Compress may cause Error: EPERM: operation not permitted open 'xxx\xxxx\xxxxx\myfile'. It`s ok with the option of compress is false.

未关闭
#175 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

主要语言
JavaScript
星标
38
派生
20
PR 合并指标
30 天内没有已合并 PR

描述

This issue is discovered from "log4js-node", The latest version of "log4js-node" has the same issue because of this dependency.

replicate step:

1. Download source code of this project.
2. Create a 'index.js' file in root of this project.
3. Input content in 'index.js' file:
```
var rollers = require('./lib/index');
var stream = new rollers.RollingFileStream('myfile', 10000, 12, {
compress: true,
});

for (let i = 0; i < 30; i++) {
// console.log(i);
stream.write(getContent());
}

stream.end();

function getContent() {
return `
{
a: {a: 1, b: 2, c: 3, d: 4, e: 5, f: 123321, g: 234432432, h: 31321, i: 23232, j: 23432423432432, k: 1123321},
b: {a: 1, b: 2, c: 3, d: 4, e: 5, f: 123321, g: 234432432, h: 31321, i: 23232, j: 23432423432432, k: 1123321},
c: {a: 1, b: 2, c: 3, d: 4, e: 5, f: 123321, g: 234432432, h: 31321, i: 23232, j: 23432423432432, k: 1123321},
......... // 70000 lines repeated
}`
}
```
4. Execute `node index.js` in terminal.
5. Open the hot file "myfile" with notebook or VSCode as you like. This step is very important. The issue will not replicate without this step.
6. Execute `node index.js` in terminal again. If no error occur, you can execute more times. The probability is very high, more than 50%.

Finally get this error in terminal:

Error: EPERM: operation not permitted, open 'D:\nodeProgram\TEST\test-streamroller\streamroller-master\myfile'
at Object.openSync (node:fs:596:3)
at Object.writeFileSync (node:fs:2322:35)
at Object.appendFileSync (node:fs:2384:6)
at RollingFileStream._renewWriteStream (D:\nodeProgram\TEST\test-streamroller\streamroller-master\lib\RollingFileWriteStream.js:312:8)
at RollingFileStream._moveOldFiles (D:\nodeProgram\TEST\test-streamroller\streamroller-master\lib\RollingFileWriteStream.js:232:10)
at async RollingFileStream._shouldRoll (D:\nodeProgram\TEST\test-streamroller\streamroller-master\lib\RollingFileWriteStream.js:170:7) {
errno: -4048,
syscall: 'open',
code: 'EPERM',
path: 'D:\\nodeProgram\\TEST\\test-streamroller\\streamroller-master\\myfile'
}

贡献指南

这个仓库没有索引到贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

从 lib/RollingFileWriteStream.js 开始,重点查看 _shouldRoll、_moveOldFiles 和 _renewWriteStream,并结合报告中的 stack trace。使用提供的 index.js、compress: true 和一个已打开的 myfile 在 Windows 上重现;当重复运行在滚动文件时不再引发 EPERM 时,即视为完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
javascript, node.js
领域
backend
Issue 类型
缺陷
难度
4/5
预计耗时
3-5 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。