Merge pull request kubernetes-client#635 from bacongobbler/remove-net…

…-keepalive fix: remove dependency on net-keepalive
bbatha · Apr 11, 2021 · 6c5a58f · 6c5a58f
2 parents a3e1501 + a3e6c0a
commit 6c5a58f
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 87 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -66,7 +66,6 @@
     "isomorphic-ws": "^4.0.1",
     "js-yaml": "^3.13.1",
     "jsonpath-plus": "^0.19.0",
-    "net-keepalive": "2.0.4",
     "openid-client": "^4.1.1",
     "request": "^2.88.0",
     "rfc4648": "^1.3.0",

diff --git a/src/watch.ts b/src/watch.ts
@@ -1,5 +1,4 @@
 import byline = require('byline');
-import keepalive = require('net-keepalive');
 import request = require('request');
 import { Duplex } from 'stream';
 import { KubeConfig } from './config';
@@ -115,8 +114,7 @@ export class Watch {
         req.on('error', doneCallOnce);
         req.on('socket', (socket) => {
             socket.setTimeout(30000);
-            socket.setKeepAlive(true);
-            keepalive.setKeepAliveInterval(socket, 30000);
+            socket.setKeepAlive(true, 30000);
         });
         stream.on('error', doneCallOnce);
         stream.on('close', () => doneCallOnce(null));