fix: proxyConfig options and defaults

Fixed up `PolykeyAgent`'s proxyConfig to align with the `Proxy`'s parameters. Updated config defaults to align as well. Fixed up failing test in `certchain.test.st` as well.
MatrixAI · Mar 29, 2022 · db3b913 · db3b913
1 parent 4402287
commit db3b913
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 10 deletions.
diff --git a/src/PolykeyAgent.ts b/src/PolykeyAgent.ts
@@ -103,8 +103,10 @@ class PolykeyAgent {
     proxyConfig?: {
       authToken?: string;
       connConnectTime?: number;
-      connTimeoutTime?: number;
-      connPingIntervalTime?: number;
+      connKeepAliveTimeoutTime?: number;
+      connEndTime?: number;
+      connPunchIntervalTime?: number;
+      connKeepAliveIntervalTime?: number;
     };
     nodeConnectionManagerConfig?: {
       connConnectTime?: number;

diff --git a/src/bin/agent/CommandStart.ts b/src/bin/agent/CommandStart.ts
@@ -29,7 +29,7 @@ class CommandStart extends CommandPolykey {
     this.addOption(binOptions.clientPort);
     this.addOption(binOptions.proxyHost);
     this.addOption(binOptions.proxyPort);
-    this.addOption(binOptions.connTimeoutTime);
+    this.addOption(binOptions.connConnectTime);
     this.addOption(binOptions.seedNodes);
     this.addOption(binOptions.network);
     this.addOption(binOptions.workers);
@@ -96,7 +96,7 @@ class CommandStart extends CommandPolykey {
           recoveryCode: recoveryCodeIn,
         },
         proxyConfig: {
-          connTimeoutTime: options.connTimeoutTime,
+          connConnectTime: options.connectionTimeout,
         },
         networkConfig: {
           clientHost: options.clientHost,

diff --git a/src/bin/utils/options.ts b/src/bin/utils/options.ts
@@ -84,12 +84,12 @@ const proxyPort = new commander.Option('-pp, --proxy-port <port>', 'Proxy Port')
   .argParser(binParsers.parsePort)
   .default(config.defaults.networkConfig.proxyPort);
 
-const connTimeoutTime = new commander.Option(
+const connConnectTime = new commander.Option(
   '--connection-timeout <ms>',
   'Timeout value for connection establishment between nodes',
 )
   .argParser(binParsers.parseInteger)
-  .default(config.defaults.proxyConfig.connTimeoutTime);
+  .default(config.defaults.proxyConfig.connConnectTime);
 
 const passwordFile = new commander.Option(
   '-pf, --password-file <path>',
@@ -164,7 +164,7 @@ export {
   clientPort,
   proxyHost,
   proxyPort,
-  connTimeoutTime,
+  connConnectTime,
   recoveryCodeFile,
   passwordFile,
   passwordNewFile,

diff --git a/src/config.ts b/src/config.ts
@@ -84,8 +84,10 @@ const config = {
     },
     proxyConfig: {
       connConnectTime: 20000,
-      connTimeoutTime: 20000,
-      connPingIntervalTime: 1000,
+      connKeepAliveTimeoutTime: 20000,
+      connEndTime: 1000,
+      connPunchIntervalTime: 1000,
+      connKeepAliveIntervalTime: 1000,
     },
     nodeConnectionManagerConfig: {
       connConnectTime: 20000,

diff --git a/tests/bin/keys/certchain.test.ts b/tests/bin/keys/certchain.test.ts
@@ -40,6 +40,6 @@ describe('certchain', () => {
     ));
     expect(exitCode).toBe(0);
     const certChainStatus = JSON.parse(stdout).rootCertChainPem;
-    expect(certChainCommand).toBe(certChainStatus);
+    expect(certChainCommand.rootPublicKeyPem).toBe(certChainStatus);
   });
 });