Skip to content

## require socket to be configured (and connected?) by socket creator #3629

@brianc

Description

@brianc

require socket to be configured (and connected?) by socket creator

I’ll try to get around to expanding on this later, but in short:

  • new config option socketOptions: {keepAlive, keepAliveInitialDelay [sic], …}
  • default socket is new net.Socket({noDelay: true, ...socketOptions}).connect(…)
  • if socket is passed, no further initialization

(see compatibility issues with Cloudflare sockets, for one; this single change might be enough to allow generic stream to be passed as socket?)

Originally posted by @charmander in #3598 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions