Explain KeymanagerServer opts merging

ChainSafe · Apr 3, 2022 · f11081e · f11081e
1 parent 31c64f5
commit f11081e
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/packages/keymanager-server/src/server.ts b/packages/keymanager-server/src/server.ts
@@ -47,9 +47,12 @@ export class KeymanagerServer {
 
   constructor(optsArg: Partial<RestApiOptions>, modules: IRestApiModules) {
     this.logger = modules.logger;
+
     // Apply opts defaults
     const opts = {
       ...restApiOptionsDefault,
+      // optsArg is a Partial type, any of its properties can be undefined. If port is set to undefined,
+      // it overrides the default port value in restApiOptionsDefault to be undefined.
       ...Object.fromEntries(Object.entries(optsArg).filter(([_, v]) => v != null)),
     };