The "fd" argument must be of type number. Received null - Erroring in recent Node versions 20 and 21 #50979

Activity

1. marco-ippolito commented on Nov 30, 2023

   

   marco-ippolito

   on Nov 30, 2023

   Last edited by marco-ippolito

   Member

   More actions

   Can you share a minimal reproducible example without 3rd party libraries and a stacktrace?
2. SamTV12345 commented on Nov 30, 2023

   

   SamTV12345

   on Nov 30, 2023

   Author

   More actions

   Sure I'll create one in the next few hours. It was thrown in the dirty db implementation so it shouldn't be too hard to track that down.
3. SamTV12345 commented on Nov 30, 2023

   

   SamTV12345

   on Nov 30, 2023

   Last edited by SamTV12345

   Author

   More actions

   @marco-ippolito Here is the example. I just put it into a test so I can easily start it from webstorm

   import {describe, it} from "vitest";
   import {TMP_PATH} from "./config";
   import fs, {createWriteStream} from "fs";
   
   describe('test-node-failure',  () =>{
   
       it('should allow .on load to chain to constructor', async function () {
           let path = "test.txt"
           let _writeStream = createWriteStream(path, {
               encoding: 'utf-8',
               flags: 'a',
           });
               let _queue = new Map();
               let key = 'key';
   
               const cbs = _queue.get(key) || [];
               _queue.set(key, cbs);
           _writeStream.cork();
           for (const [key, _] of _queue) {
               _queue.delete(key);
               const data = `${JSON.stringify({key, val: "test"})}\n`;
               _writeStream.write(data, (err) => {
                   console.log(err)
               });
           }
           _writeStream.uncork();
           });
   });
4. SamTV12345 commented on Nov 30, 2023

   

   SamTV12345

   on Nov 30, 2023

   Last edited by SamTV12345

   Author

   More actions

   Stacktrace looks like this:

   TypeError: The "fd" argument must be of type number. Received null
       at Object.write (node:fs:834:8)
       at WriteStream.writeAll (node:internal/fs/streams:418:13)
       at WriteStream._write (node:internal/fs/streams:477:12)
       at doWrite (node:internal/streams/writable:590:12)
       at clearBuffer (node:internal/streams/writable:773:7)
       at WriteStream.Writable.uncork (node:internal/streams/writable:523:7)
       at C:\Users\user\WebstormProjects\dirty_db\test\test-minimal.test.ts:25:22
       at file:///C:/Users/user/WebstormProjects/dirty_db/node_modules/@vitest/runner/dist/index.js:135:14
       at file:///C:/Users/user/WebstormProjects/dirty_db/node_modules/@vitest/runner/dist/index.js:58:26
       at runTest (file:///C:/Users/user/WebstormProjects/dirty_db/node_modules/@vitest/runner/dist/index.js:663:17)
   
5. nodejs-github-bot

   closed this as completedin #50994on Dec 7, 2023
6. nodejs-github-bot

   added a commit that references this issue on Dec 7, 2023

   stream: fix fd is null when calling clearBuffer

   ...

   639c366
7. RafaelGSS

   added a commit that references this issue on Dec 15, 2023

   stream: fix fd is null when calling clearBuffer

   ...

   5112306
8. richardlau

   added a commit that references this issue on Mar 25, 2024

   stream: fix fd is null when calling clearBuffer

   ...

   8750070