net: write(cb) not called if destroy():ed before 'connect' #30841

Activity

1. ronag

   changed the title [-]net: write(cb) not called if destroyed[/-] [+]net: `write(cb)` not called if `destroy()`:ed before 'connect'[/+] on Dec 7, 2019
2. ronag

   changed the title [-]net: `write(cb)` not called if `destroy()`:ed before 'connect'[/-] [+]net: `write(cb)` not called if `destroy()`:ed before `'connect'`[/+] on Dec 7, 2019
3. lpinca

   added

   netIssues and PRs related to the net subsystem.Issues and PRs related to the net subsystem.

   on Dec 7, 2019
4. ronag commented on Jan 3, 2020

   

   ronag

   on Jan 3, 2020

   MemberAuthor

   More actions

   This is fixed
5. ronag

   closed this as completedon Jan 3, 2020
6. lpinca commented on Jan 5, 2020

   

   lpinca

   on Jan 5, 2020

   Member

   More actions

   If the expectation is that callback is called this is not fixed.

   const assert = require('assert');
   const net = require('net');
   
   const socket = new net.Socket();
   
   socket.connect({
     port: 80,
     host: 'example.com',
     lookup() {}
   });
   
   assert(socket.connecting);
   
   let called = false;
   
   socket.write('foo', function() {
     called = true;
   });
   
   socket.destroy();
   
   process.on('exit', function() {
     assert(called);
   });

   👍React with 👍1Reacted by Robert Nagy
7. lpinca

   reopened this on Jan 5, 2020
8. ronag commented on Jan 5, 2020

   

   ronag

   on Jan 5, 2020

   Last edited by ronag

   MemberAuthor

   More actions

   Yea, we should add a test for that.

   This seems related to the whole constructing state which in this case never gets resolved, i.e. _write never finishes https://github.com/nodejs/node/blob/master/lib/net.js#L763. I can dig into this once/if #29656 lands.
9. lpinca commented on Jan 5, 2020

   

   lpinca

   on Jan 5, 2020

   Last edited by lpinca

   Member

   More actions

   yes, _writeGeneric() is never called as 'connect' is never emitted.

   👍React with 👍1Reacted by Robert Nagy
10. ronag commented on Jan 6, 2020

    

    ronag

    on Jan 6, 2020

    MemberAuthor

    More actions

    pending resolution to discussion in #31179
11. ronag commented on Apr 19, 2020

    

    ronag

    on Apr 19, 2020

    MemberAuthor

    More actions

    'use strict';
    
    const common = require('../common');
    const assert = require('assert');
    const net = require('net');
    
    const socket = new net.Socket();
    
    socket.connect({
      port: 80,
      host: 'example.com',
      lookup() {}
    });
    
    assert(socket.connecting);
    
    socket.write('foo', common.mustCall());
    socket.destroy();
12. ronag commented on Apr 19, 2020

    

    ronag

    on Apr 19, 2020

    MemberAuthor

    More actions

    This requires #29656 in order to be fully resolved.
13. ronag

    added

    confirmed-bugIssues and PRs for confirmed bugs.Issues and PRs for confirmed bugs.

    on Apr 19, 2020