@requestly/requestly-proxy 1.3.10 → 1.3.12

package/dist/components/proxy-middleware/index.js

@@ -146,6 +146,7 @@ class ProxyMiddlewareManager {
                     let pre_final_body = parsedBody || body.toString("utf8");
                     ctx.rq.set_original_request({ body: pre_final_body });
                     ctx.rq_request_body = pre_final_body;
+                    let request_rule_applied = false;
                     if (parsedBody && constants_1.RQ_INTERCEPTED_CONTENT_TYPES.includes(contentType)) {
                         // Do modifications, if any
                         const { action_result_objs, continue_request } = await rules_middleware.on_request_end(ctx);
@@ -153,9 +154,15 @@ class ProxyMiddlewareManager {
                             logger_middleware.send_network_log(ctx, rules_middleware.action_result_objs, requestly_core_1.CONSTANTS.REQUEST_STATE.COMPLETE);
                             return;
                         }
+                        request_rule_applied = action_result_objs === null || action_result_objs === void 0 ? void 0 : action_result_objs.some((obj) => { var _a; return ((_a = obj === null || obj === void 0 ? void 0 : obj.action) === null || _a === void 0 ? void 0 : _a.action) === constants_1.RULE_ACTION.MODIFY_REQUEST; });
+                    }
+                    if (request_rule_applied) {
+                        ctx.proxyToServerRequest.write(ctx.rq_request_body);
+                    }
+                    else {
+                        // If no modifications, write the original request body buffer so that we don't mess up during decoding
+                        ctx.proxyToServerRequest.write(body);
                     }
-                    // Use the updated request
-                    ctx.proxyToServerRequest.write(ctx.rq_request_body);
                     ctx.rq.set_final_request({ body: ctx.rq_request_body });
                     return callback();
                 });

package/dist/components/proxy-middleware/middlewares/logger_middleware.js

@@ -71,7 +71,7 @@ class LoggerMiddleware {
             const rqLog = {
                 id: ctx.uuid,
                 timestamp: Math.floor(Date.now() / 1000),
-                finalHar: (0, harObectCreator_1.createHar)(ctx.rq.final_request.headers, ctx.rq.final_request.method, protocol, ctx.rq.final_request.host, ctx.rq.final_request.path, ctx.rq.final_request.body, ctx.rq.final_response.status_code, ctx.rq.final_response.body, ctx.rq.final_response.headers || {}, ctx.rq.final_request.query_params),
+                finalHar: (0, harObectCreator_1.createHar)(ctx.rq.final_request.headers, ctx.rq.final_request.method, protocol, (0, utils_1.getHost)(ctx), ctx.rq.final_request.path, ctx.rq.final_request.body, ctx.rq.final_response.status_code, ctx.rq.final_response.body, ctx.rq.final_response.headers || {}, ctx.rq.final_request.query_params),
                 requestShellCurl: this.generate_curl_from_har((_b = (_a = ctx === null || ctx === void 0 ? void 0 : ctx.rq) === null || _a === void 0 ? void 0 : _a.final_request) === null || _b === void 0 ? void 0 : _b.requestHarObject), // TODO: Move this to client side
                 actions: (0, utils_1.get_success_actions_from_action_results)(action_result_objs),
                 consoleLogs: (_c = ctx === null || ctx === void 0 ? void 0 : ctx.rq) === null || _c === void 0 ? void 0 : _c.consoleLogs,

package/dist/components/proxy-middleware/rule_action_processor/utils.d.ts

@@ -9,4 +9,5 @@ export function build_post_process_data(status_code: any, headers: any, body: an
     body: any;
 };
 export function get_success_actions_from_action_results(action_result_objs?: any[]): any[];
+export function getHost(ctx: any): any;
 export function get_file_contents(file_path: any): string;

package/dist/components/proxy-middleware/rule_action_processor/utils.js

@@ -3,7 +3,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.get_file_contents = exports.get_success_actions_from_action_results = exports.build_post_process_data = exports.build_action_processor_response = void 0;
+exports.get_file_contents = exports.getHost = exports.get_success_actions_from_action_results = exports.build_post_process_data = exports.build_action_processor_response = void 0;
 const fs_1 = __importDefault(require("fs"));
 const build_action_processor_response = (action, success = false, post_process_data = null) => {
     let resp = {
@@ -35,6 +35,18 @@ const get_success_actions_from_action_results = (action_result_objs = []) => {
     return success_action_results_objs.map((obj) => obj.action);
 };
 exports.get_success_actions_from_action_results = get_success_actions_from_action_results;
+const getHost = (ctx) => {
+    const finalHost = ctx.rq.final_request.host;
+    const port = ctx.rq.final_request.port;
+    const protocol = ctx.isSSL ? "https" : "http";
+    const standardPort = protocol === "https" ? 443 : 80;
+    // Only append port if it's non-standard and not already in the host
+    if (port && port !== standardPort && !finalHost.includes(':')) {
+        return `${finalHost}:${port}`;
+    }
+    return finalHost;
+};
+exports.getHost = getHost;
 const get_file_contents = (file_path) => {
     return fs_1.default.readFileSync(file_path, "utf-8");
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@requestly/requestly-proxy",
-  "version": "1.3.10",
+  "version": "1.3.12",
   "description": "Proxy that gives superpowers to all the Requestly clients",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -26,7 +26,7 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "@requestly/requestly-core": "^1.1.0",
+    "@requestly/requestly-core": "1.1.1",
     "@sentry/browser": "^8.33.1",
     "async": "^3.2.5",
     "axios": "^1.7.2",