dce-expresskit 5.0.4 → 5.0.5

package/lib/helpers/visitEndpointOnAnotherServer/index.js

@@ -60,22 +60,32 @@ var sendServerToServerRequest_1 = __importDefault(require("./sendServerToServerR
  * @returns the body of the response from the other server
  */
 var visitEndpointOnAnotherServer = function (opts) { return __awaiter(void 0, void 0, void 0, function () {
-    var response, body;
-    return __generator(this, function (_a) {
-        switch (_a.label) {
-            case 0: return [4 /*yield*/, (0, sendServerToServerRequest_1.default)({
-                    path: opts.path,
-                    host: opts.host,
-                    method: opts.method,
-                    params: opts.params,
-                    responseType: opts.responseType,
-                    dceKitCrossServerCredentials: opts.dceKitCrossServerCredentials,
-                })];
+    var sanitizedParams, response, body;
+    var _a;
+    return __generator(this, function (_b) {
+        switch (_b.label) {
+            case 0:
+                sanitizedParams = {};
+                Object.keys((_a = opts.params) !== null && _a !== void 0 ? _a : {}).forEach(function (key) {
+                    var _a;
+                    var value = (_a = opts.params) === null || _a === void 0 ? void 0 : _a[key];
+                    if (value !== undefined) {
+                        sanitizedParams[key] = value;
+                    }
+                });
+                return [4 /*yield*/, (0, sendServerToServerRequest_1.default)({
+                        path: opts.path,
+                        host: opts.host,
+                        method: opts.method,
+                        params: sanitizedParams,
+                        responseType: opts.responseType,
+                        dceKitCrossServerCredentials: opts.dceKitCrossServerCredentials,
+                    })];
             case 1:
-                response = _a.sent();
+                response = _b.sent();
                 // Check for failure
                 if (!response || !response.body) {
-                    throw new dce_commonkit_1.ErrorWithCode('We didn\'t get a response from the other server. Please check the network between the two connection.', dce_commonkit_1.CommonKitErrorCode.NoResponse);
+                    throw new dce_commonkit_1.ErrorWithCode('We didn\'t get a response from the other server. Please check the network between the two servers.', dce_commonkit_1.CommonKitErrorCode.NoResponse);
                 }
                 if (!response.body.success) {
                     // Other errors

package/lib/helpers/visitEndpointOnAnotherServer/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/helpers/visitEndpointOnAnotherServer/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,uBAAuB;AACvB,+CAGuB;AAEvB,sBAAsB;AACtB,0FAAoE;AAEpE,4EAA4E;AAC5E,4EAA4E;AAC5E,4EAA4E;AAE5E;;;;;;;;;;;;GAYG;AACH,IAAM,4BAA4B,GAAG,UACnC,IAOC;;;;oBAGgB,qBAAM,IAAA,mCAAyB,EAAC;oBAC/C,IAAI,EAAE,IAAI,CAAC,IAAI;oBACf,IAAI,EAAE,IAAI,CAAC,IAAI;oBACf,MAAM,EAAE,IAAI,CAAC,MAAM;oBACnB,MAAM,EAAE,IAAI,CAAC,MAAM;oBACnB,YAAY,EAAE,IAAI,CAAC,YAAY;oBAC/B,4BAA4B,EAAE,IAAI,CAAC,4BAA4B;iBAChE,CAAC,EAAA;;gBAPI,QAAQ,GAAG,SAOf;gBAEF,oBAAoB;gBACpB,IAAI,CAAC,QAAQ,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC;oBAChC,MAAM,IAAI,6BAAa,CACrB,uGAAuG,EACvG,kCAAkB,CAAC,UAAU,CAC9B,CAAC;gBACJ,CAAC;gBACD,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,OAAO,EAAE,CAAC;oBAC3B,eAAe;oBACf,MAAM,IAAI,6BAAa,CACrB,CACE,QAAQ,CAAC,IAAI,CAAC,OAAO;2BAClB,qDAAqD,CACzD,EACD,CACE,QAAQ,CAAC,IAAI,CAAC,IAAI;2BACf,kCAAkB,CAAC,MAAM,CAC7B,CACF,CAAC;gBACJ,CAAC;gBAGO,IAAI,GAAK,QAAQ,CAAC,IAAI,KAAlB,CAAmB;gBAE/B,SAAS;gBACT,sBAAO,IAAI,EAAC;;;KACb,CAAC;AAEF,kBAAe,4BAA4B,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/helpers/visitEndpointOnAnotherServer/index.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,uBAAuB;AACvB,+CAGuB;AAEvB,sBAAsB;AACtB,0FAAoE;AAEpE,4EAA4E;AAC5E,4EAA4E;AAC5E,4EAA4E;AAE5E;;;;;;;;;;;;GAYG;AACH,IAAM,4BAA4B,GAAG,UACnC,IAOC;;;;;;gBAGK,eAAe,GAA6B,EAAE,CAAC;gBACrD,MAAM,CAAC,IAAI,CAAC,MAAA,IAAI,CAAC,MAAM,mCAAI,EAAE,CAAC,CAAC,OAAO,CAAC,UAAC,GAAG;;oBACzC,IAAM,KAAK,GAAG,MAAA,IAAI,CAAC,MAAM,0CAAG,GAAG,CAAC,CAAC;oBACjC,IAAI,KAAK,KAAK,SAAS,EAAE,CAAC;wBACxB,eAAe,CAAC,GAAG,CAAC,GAAG,KAAK,CAAC;oBAC/B,CAAC;gBACH,CAAC,CAAC,CAAC;gBAGc,qBAAM,IAAA,mCAAyB,EAAC;wBAC/C,IAAI,EAAE,IAAI,CAAC,IAAI;wBACf,IAAI,EAAE,IAAI,CAAC,IAAI;wBACf,MAAM,EAAE,IAAI,CAAC,MAAM;wBACnB,MAAM,EAAE,eAAe;wBACvB,YAAY,EAAE,IAAI,CAAC,YAAY;wBAC/B,4BAA4B,EAAE,IAAI,CAAC,4BAA4B;qBAChE,CAAC,EAAA;;gBAPI,QAAQ,GAAG,SAOf;gBAEF,oBAAoB;gBACpB,IAAI,CAAC,QAAQ,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC;oBAChC,MAAM,IAAI,6BAAa,CACrB,oGAAoG,EACpG,kCAAkB,CAAC,UAAU,CAC9B,CAAC;gBACJ,CAAC;gBACD,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,OAAO,EAAE,CAAC;oBAC3B,eAAe;oBACf,MAAM,IAAI,6BAAa,CACrB,CACE,QAAQ,CAAC,IAAI,CAAC,OAAO;2BAClB,qDAAqD,CACzD,EACD,CACE,QAAQ,CAAC,IAAI,CAAC,IAAI;2BACf,kCAAkB,CAAC,MAAM,CAC7B,CACF,CAAC;gBACJ,CAAC;gBAGO,IAAI,GAAK,QAAQ,CAAC,IAAI,KAAlB,CAAmB;gBAE/B,SAAS;gBACT,sBAAO,IAAI,EAAC;;;KACb,CAAC;AAEF,kBAAe,4BAA4B,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dce-expresskit",
-  "version": "5.0.4",
+  "version": "5.0.5",
   "description": "Shared functions, helpers, and tools for Harvard DCE Express-based servers",
   "main": "./lib/index.js",
   "types": "./lib/index.d.ts",

package/src/helpers/visitEndpointOnAnotherServer/index.ts

@@ -34,12 +34,21 @@ const visitEndpointOnAnotherServer = async (
     dceKitCrossServerCredentials?: string,
   },
 ): Promise<any> => {
+  // Sanitize params: remove undefined values
+  const sanitizedParams: { [key in string]: any } = {};
+  Object.keys(opts.params ?? {}).forEach((key) => {
+    const value = opts.params?.[key];
+    if (value !== undefined) {
+      sanitizedParams[key] = value;
+    }
+  });
+
   // Send the request
   const response = await sendServerToServerRequest({
     path: opts.path,
     host: opts.host,
     method: opts.method,
-    params: opts.params,
+    params: sanitizedParams,
     responseType: opts.responseType,
     dceKitCrossServerCredentials: opts.dceKitCrossServerCredentials,
   });
@@ -47,7 +56,7 @@ const visitEndpointOnAnotherServer = async (
   // Check for failure
   if (!response || !response.body) {
     throw new ErrorWithCode(
-      'We didn\'t get a response from the other server. Please check the network between the two connection.',
+      'We didn\'t get a response from the other server. Please check the network between the two servers.',
       CommonKitErrorCode.NoResponse,
     );
   }