writable.end(cb), the cb is never called

[a1exwang]

Version

16.12.0

Platform

Linux alex-p330 5.13.0-19-generic #19-Ubuntu SMP Thu Oct 7 21:58:00 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Subsystem

stream

What steps will reproduce the bug?

touch testfile
chmod 400 testfile

2. create test.js with the following content

const stream = require("stream");
const fs = require("fs");

const w = fs.createWriteStream("testfile");
w.write("test")
stream.finished(w, (e) => {
  console.log('finished(): ' + e)
  w.end(() => {
    console.log('end()')
  })
})

setTimeout(() => { console.log("timeout"); }, 10 * 1000)

3. node test.js and wait 10 seconds

How often does it reproduce? Is there a required condition?

always

What is the expected behavior?

finished(): Error: EACCES: permission denied, open 'testfile'
end()
timeout

What do you see instead?

finished(): Error: EACCES: permission denied, open 'testfile'
timeout

Additional information

No response

[ronag]

Can you confirm whether you have the same error on v17?

[ronag]

I'm unable to reproduce on master.

[targos]

@ronag do you know what fixed it? Can we backport it to LTS?

[alex-petrov-vt]

I get the expected behavior when running this on node v16.12.0, v16.13.0 and v17.1.0 (both locally and in docker). I do get the same issue as @a1exwang when running on node v14.17.2 but I'm not sure if that's the expected behavior for that version.

Just wanted to leave this info here in case someone finds it useful

[MoonBall]

I get the expected behavior when running this on code of master branch. My OS is masOS Monterey.