Merge pull request #2162 from Hyperkid123/content-length-issues

fix(proxy): deflate gziped responses
RedHatInsights · Jan 28, 2025 · bf54487 · bf54487
2 parents 174ffcd + a4b9409
commit bf54487
Show file tree

Hide file tree

Showing 15 changed files with 43 additions and 29 deletions.
diff --git a/packages/advisor-components/project.json b/packages/advisor-components/project.json
@@ -83,7 +83,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/chrome/project.json b/packages/chrome/project.json
@@ -57,7 +57,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/components/project.json b/packages/components/project.json
@@ -82,7 +82,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/config-utils/project.json b/packages/config-utils/project.json
@@ -32,7 +32,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/config-utils/src/proxy.ts b/packages/config-utils/src/proxy.ts
@@ -234,12 +234,6 @@ const proxy = ({
       secure: false,
       changeOrigin: true,
       autoRewrite: true,
-      onProxyReq: (proxyReq, req) => {
-        if (isInterceptAbleRequest(req.url)) {
-          // necessary to avoid gzip encoding and issues with parsing the json body
-          proxyReq.setHeader('accept-encoding', 'gzip;q=0,deflate,sdch');
-        }
-      },
       onProxyRes: (proxyRes, req, res) => {
         // this should reading the aggregated bundles filed generated from chrome service
         // The functionality is disabled until the interceptor is ready
@@ -319,15 +313,21 @@ const proxy = ({
 
         return null;
       },
-      ...(agent && {
-        agent,
-        headers: {
-          // Staging Akamai CORS gives 403s for non-GET requests from non-origin hosts
-          Host: target.replace('https://', ''),
-          // Origin format is protocol://hostname:port only, remove any trailing slash
-          Origin: target.replace(/\/$/, ''),
-        },
-      }),
+      ...(agent
+        ? {
+            agent,
+            headers: {
+              // Set FEO gzip headers here to prevent issues with setting headers after response was closed
+              'accept-encoding': 'gzip;q=0,deflate,sdch',
+              // Staging Akamai CORS gives 403s for non-GET requests from non-origin hosts
+              Host: target.replace('https://', ''),
+              // Origin format is protocol://hostname:port only, remove any trailing slash
+              Origin: target.replace(/\/$/, ''),
+            },
+          }
+        : {
+            'accept-encoding': 'gzip;q=0,deflate,sdch',
+          }),
     });
   }
 

diff --git a/packages/config/project.json b/packages/config/project.json
@@ -40,7 +40,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/eslint-config/project.json b/packages/eslint-config/project.json
@@ -27,7 +27,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/notifications/project.json b/packages/notifications/project.json
@@ -72,7 +72,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/remediations/project.json b/packages/remediations/project.json
@@ -73,7 +73,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/rule-components/project.json b/packages/rule-components/project.json
@@ -73,7 +73,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/testing/project.json b/packages/testing/project.json
@@ -65,7 +65,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/translations/project.json b/packages/translations/project.json
@@ -84,7 +84,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/tsc-transform-imports/project.json b/packages/tsc-transform-imports/project.json
@@ -32,7 +32,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/types/project.json b/packages/types/project.json
@@ -26,7 +26,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },

diff --git a/packages/utils/project.json b/packages/utils/project.json
@@ -72,7 +72,8 @@
       "options": {
         "push": true,
         "preset": "conventionalcommits",
-        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]"
+        "commitMessageFormat": "chore: bump {projectName} to {version} [skip ci]",
+        "trackDeps": true
       },
       "dependsOn": ["^version"]
     },