@requestly/requestly-proxy 1.3.11 → 1.3.13

package/dist/components/proxy-middleware/constants.d.ts

@@ -9,3 +9,4 @@ export namespace RULE_ACTION {
     let MODIFY_REQUEST: string;
 }
 export const RQ_INTERCEPTED_CONTENT_TYPES: string[];
+export const RQ_INTERCEPTED_CONTENT_TYPES_REGEX: RegExp;

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

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.RQ_INTERCEPTED_CONTENT_TYPES = exports.RULE_ACTION = void 0;
+exports.RQ_INTERCEPTED_CONTENT_TYPES_REGEX = exports.RQ_INTERCEPTED_CONTENT_TYPES = exports.RULE_ACTION = void 0;
 exports.RULE_ACTION = {
     REDIRECT: "redirect",
     MODIFY_HEADERS: "modify_headers",
@@ -22,3 +22,15 @@ exports.RQ_INTERCEPTED_CONTENT_TYPES = [
     "application/json",
     "application/manifest+json"
 ];
+exports.RQ_INTERCEPTED_CONTENT_TYPES_REGEX = new RegExp([
+    'text/html', // HTML documents
+    'text/plain', // Plain text
+    'text/javascript', // JavaScript files
+    'application/javascript', // JavaScript (standard MIME)
+    'application/x-javascript', // JavaScript (legacy)
+    'text/css', // CSS files
+    'application/css', // CSS (alternative)
+    'application/json', // JSON data
+    'application/.+\\+json', // JSON-based media types (including vendor-specific like application/vnd.*)
+].join('|'), 'i');
+// console.log(RQ_INTERCEPTED_CONTENT_TYPES_REGEX);

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

@@ -147,7 +147,7 @@ class ProxyMiddlewareManager {
                     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)) {
+                    if (parsedBody && constants_1.RQ_INTERCEPTED_CONTENT_TYPES_REGEX.test(contentType)) {
                         // Do modifications, if any
                         const { action_result_objs, continue_request } = await rules_middleware.on_request_end(ctx);
                         if (!continue_request) {
@@ -189,13 +189,21 @@ class ProxyMiddlewareManager {
                     ctx.rq_response_body = body;
                     ctx.rq_parsed_response_body = parsedBody;
                     ctx.rq_response_status_code = (0, proxy_ctx_helper_1.getResponseStatusCode)(ctx);
-                    if (constants_1.RQ_INTERCEPTED_CONTENT_TYPES.includes(contentType) && parsedBody) {
+                    if (constants_1.RQ_INTERCEPTED_CONTENT_TYPES_REGEX.test(contentType) && parsedBody) {
                         ctx.rq_response_body = parsedBody;
                         ctx.rq.set_original_response({ body: parsedBody });
                     }
                     const { action_result_objs, continue_request } = await rules_middleware.on_response_end(ctx);
                     const statusCode = ctx.rq_response_status_code || (0, proxy_ctx_helper_1.getResponseStatusCode)(ctx);
-                    ctx.proxyToClientResponse.writeHead(statusCode, http_1.default.STATUS_CODES[statusCode], (0, proxy_ctx_helper_1.getResponseHeaders)(ctx));
+                    const responseHeaders = (0, proxy_ctx_helper_1.getResponseHeaders)(ctx);
+                    // For 204/304/1xx, remove content headers to prevent errors
+                    if (statusCode === 204 || statusCode === 304 || (statusCode >= 100 && statusCode < 200)) {
+                        delete responseHeaders['content-length'];
+                        delete responseHeaders['Content-Length'];
+                        delete responseHeaders['transfer-encoding'];
+                        delete responseHeaders['Transfer-Encoding'];
+                    }
+                    ctx.proxyToClientResponse.writeHead(statusCode, http_1.default.STATUS_CODES[statusCode], responseHeaders);
                     ctx.proxyToClientResponse.write(ctx.rq_response_body);
                     ctx.rq.set_final_response({
                         ...(0, proxy_ctx_helper_1.get_response_options)(ctx),

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/processors/modify_response_processor.js

@@ -63,7 +63,7 @@ const process_modify_response_action = async (action, ctx) => {
         action.responseType === requestly_core_1.CONSTANTS.RESPONSE_BODY_TYPES.CODE) {
         const contentTypeHeader = (0, proxy_ctx_helper_1.getResponseContentTypeHeader)(ctx);
         const contentType = (0, http_helpers_1.getContentType)(contentTypeHeader);
-        if (constants_1.RQ_INTERCEPTED_CONTENT_TYPES.includes(contentType) || contentType == null) {
+        if (constants_1.RQ_INTERCEPTED_CONTENT_TYPES_REGEX.test(contentType) || contentType == null) {
             await modify_response_using_code(action, ctx);
             delete_breaking_headers(ctx);
             return (0, utils_1.build_action_processor_response)(action, true);

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.11",
+  "version": "1.3.13",
   "description": "Proxy that gives superpowers to all the Requestly clients",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -9,11 +9,12 @@
     "test": "echo \"Error: no test specified\" && exit 1",
     "dev": "nodemon src/test.ts",
     "build": "tsc",
-    "watch": "tsc --watch"
+    "watch": "tsc --watch",
+    "release": "release-it"
   },
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/requestly/rq-proxy.git"
+    "url": "git+https://github.com/requestly/requestly-proxy.git"
   },
   "exports": {
     ".": {
@@ -45,10 +46,12 @@
     "ws": "^8.16.0"
   },
   "devDependencies": {
+    "@release-it/conventional-changelog": "^8.0.2",
     "@types/har-format": "^1.2.15",
     "@types/node": "^20.12.12",
     "ajv": "^8.13.0",
     "nodemon": "^3.1.0",
+    "release-it": "^17.10.0",
     "type-fest": "^2.12.2",
     "typescript": "^5.6.3"
   },