fix: [fix/custom-headers]: merging of custom headers with proxying re…

…quest headers (#2)

bin/corsyusha.js

@@ -72,15 +72,15 @@ const main = async () => {
 
     const normalizedServerLogging = [null, 'true', '1', 'yes', true, 1].includes(serverLogging)
 
-    const normalizedHeaders = JSON.parse(headers)
+    const parsedHeaders = JSON.parse(headers)
 
     await corsyusha({
       url,
       port,
       host,
       urlSection,
       serverLogging: normalizedServerLogging,
-      headers: normalizedHeaders,
+      headers: parsedHeaders,
     })
   } catch (error) {
     console.error(error)

lib/middleware.js

@@ -17,16 +17,21 @@ module.exports = ({
 
   try {
 
-    const requestOptions = {
-      url: requestUrl,
-      ...(
-        typeof headers === 'object' && Object.keys(headers).length ?
-          { headers } :
-          {}
-      ),
+    if (typeof headers === 'object') {
+      const normalizedHeaders = Object.keys(headers).reduce((result, h) => ({
+        ...result,
+        [h.toLowerCase()]: headers[h],
+      }), {})
+
+      req.headers = {
+        ...normalizedHeaders,
+        ...req.headers,
+      }
     }
 
-    req.pipe(request(requestOptions)).pipe(res)
+    logger.info(`${'Proxied request merged headers'.blue} ${JSON.stringify(req.headers).green}`)
+
+    req.pipe(request(requestUrl)).pipe(res)
 
   } catch (error) {
     logger.error(error)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "corsyusha",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Simple and fast (built on top of fastify) proxy to bypass CORS issues during prototyping against existing APIs without having to worry about CORS",
   "keywords": [
     "cors",