René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"net.connect options noDelay, keepAlive, keepAliveInitialDelay should be in the constructor instead","articleBody":"### Affected URL(s)\n\nhttps://nodejs.org/api/net.html\n\n### Description of the problem\n\nThe documentation for the `connect` method on `net.Socket` (https://nodejs.org/api/net.html#socketconnectoptions-connectlistener) says that there are options `noDelay`, `keepAlive`, `keepAliveInitialDelay`. However, by looking at the source code, these options are in fact instead for the constructor of `new net.Socket`.\n\nTherefore, this code won't work (the last three options will be ignored):\n```js\nlet s = new net.Socket();\n\ns.connect({\n    port: 80,\n    host: \"[some host]\",\n    noDelay: true,\n    keepAlive: true,\n    keepAliveInitialDelay: 60000});\n```\n\nMoving them to the options passed to the constructor instead will make it work.\n\nThis issue can easily be verified with `strace` on Linux.\n\nI suggest to fix the documentation to match the actual behaviour rather than changing the implementation to match the documentation. Looking at what options are available for the constructor vs. the connect method, it makes more sense to have these options in the constructor, since the connect method only have options regarding to the actual connection attempt (such as what host, port, ip family etc. to use for the connection attempt), while the socket constructor takes options to use for the socket itself, such as if half-open should be enabled etc.\n\nIt seems this issue has been present ever since these new options were added in https://github.com/nodejs/node/pull/41310. Note that if you use the `net.createConnection` factory method, you won't notice the issue since the same options are passed to both methods.","author":{"url":"https://github.com/Emill","@type":"Person","name":"Emill"},"datePublished":"2025-09-09T12:03:24.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/59823/node/issues/59823"}