@requestly/requestly-proxy 1.1.1 → 1.1.3

package/dist/components/proxy-middleware/helpers/redirectHelper.js

@@ -40,14 +40,13 @@ const parser = require("ua-parser-js");
 const Sentry = __importStar(require("@sentry/browser"));
 const https_1 = __importDefault(require("https"));
 const http_1 = __importDefault(require("http"));
-const willCreateMixedResponse = (ctx, destinationUrl) => {
-    var _a, _b, _c, _d;
+const willCreateMixedResponseThatCanBeHandled = (ctx, destinationUrl) => {
+    var _a, _b, _c;
     let user_agent_str = null;
     user_agent_str = (_a = ctx === null || ctx === void 0 ? void 0 : ctx.clientToProxyRequest) === null || _a === void 0 ? void 0 : _a.headers["user-agent"];
-    console.log("handling mixed response. i got headers", (_b = ctx === null || ctx === void 0 ? void 0 : ctx.clientToProxyRequest) === null || _b === void 0 ? void 0 : _b.headers);
-    const user_agent = (_d = (_c = parser(user_agent_str)) === null || _c === void 0 ? void 0 : _c.browser) === null || _d === void 0 ? void 0 : _d.name;
+    const user_agent = (_c = (_b = parser(user_agent_str)) === null || _b === void 0 ? void 0 : _b.browser) === null || _c === void 0 ? void 0 : _c.name;
     const LOCAL_DOMAINS = ["localhost", "127.0.0.1"];
-    return ctx.isSSL && destinationUrl.includes("http") && (user_agent === "Safari" ||
+    return ctx.isSSL && destinationUrl.includes("http:") && (user_agent === "Safari" ||
         !LOCAL_DOMAINS.some((domain) => destinationUrl.includes(domain)));
 };
 const canPreserveCookie = (ctx, destinationUrl) => {
@@ -82,7 +81,7 @@ const canPreserveCookie = (ctx, destinationUrl) => {
 };
 const shouldMakeExternalRequest = (ctx, action) => {
     return ((action.preserveCookie && canPreserveCookie(ctx, action.url)) ||
-        willCreateMixedResponse(ctx, action.url));
+        willCreateMixedResponseThatCanBeHandled(ctx, action.url));
 };
 exports.shouldMakeExternalRequest = shouldMakeExternalRequest;
 function makeRequest(requestOptions) {

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

@@ -15,6 +15,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.MIDDLEWARE_TYPE = void 0;
 // Move Proxy code to this package
 var net = require("net");
+const http_1 = __importDefault(require("http"));
 const proxy_1 = __importDefault(require("../../lib/proxy"));
 const proxy_ctx_helper_1 = require("./helpers/proxy_ctx_helper");
 const rules_middleware_1 = __importDefault(require("./middlewares/rules_middleware"));
@@ -149,7 +150,8 @@ class ProxyMiddlewareManager {
                             // Remove some conflicting headers like content-length, if any
                             delete (0, proxy_ctx_helper_1.getResponseHeaders)(ctx)["content-length"];
                         }
-                        ctx.proxyToClientResponse.writeHead(ctx.rq_response_status_code || (0, proxy_ctx_helper_1.getResponseStatusCode)(ctx), (0, proxy_ctx_helper_1.getResponseHeaders)(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));
                         ctx.proxyToClientResponse.write(ctx.rq_response_body);
                         ctx.rq.set_final_response(Object.assign(Object.assign({}, (0, proxy_ctx_helper_1.get_response_options)(ctx)), { status_code: ctx.rq_response_status_code || (0, proxy_ctx_helper_1.getResponseStatusCode)(ctx), body: ctx.rq_response_body }));
                         logger_middleware.send_network_log(ctx, rules_middleware.action_result_objs, requestly_core_1.CONSTANTS.REQUEST_STATE.COMPLETE);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@requestly/requestly-proxy",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "description": "Proxy that gives superpowers to all the Requestly clients",
   "main": "dist/index.js",
   "scripts": {