feat: onReq now takes (req, proxyReqOptions), instead of (req, headers)

Breaks onReq signature

Author: ronag

README.md

@@ -69,7 +69,7 @@ server.on('request', (req, res) => {
   proxy.web(req, res, {
     hostname: 'localhost'
     port: 9000,
-    onReq: (req, headers) => {
+    onReq: (req, { headers }) => {
       headers['x-forwarded-for'] = req.socket.remoteAddress
       headers['x-forwarded-proto'] = req.socket.encrypted ? 'https' : 'http'
       headers['x-forwarded-host'] = req.headers['host']
@@ -102,9 +102,9 @@ server.on('request', (req, res) => {
   - `timeout`: incoming request timeout
   - `proxyTimeout`: proxy request timeout
   - `proxyName`: proxy name used for **Via** header
-  - `onReq(req, headers)`: called before proxy request
+  - `onReq(req, options)`: called before proxy request
     - `req`: `http.IncomingMessage` or `http2.Http2ServerRequest`
-    - `headers`: `http.ClientRequest.headers`
+    - `options`: options passed to `http.ClientRequest`
   - `onRes(req, res)`: called before proxy response
     - `req`: `http.IncomingMessage` or `http2.Http2ServerRequest`
     - `res`: `http.ServerResponse` or `http2.Http2ServerResponse`

index.js

@@ -110,23 +110,25 @@ function impl (req, resOrSocket, headOrNil, {
       }
     }
 
+    const options = {
+      method: req.method,
+      hostname,
+      port,
+      path: req.url,
+      headers,
+      timeout: proxyTimeout
+    }
+
     if (onReq) {
-      onReq(req, headers)
+      onReq(req, options)
     }
 
     // NOTE http2.Http2ServerRequest doesn't forward stream errors.
     const incoming = req.stream || req
 
     incoming.on('error', onError)
 
-    return proxy(req, resOrSocket, {
-      method: req.method,
-      hostname,
-      port,
-      path: req.url,
-      headers,
-      timeout: proxyTimeout
-    }, onRes, onError)
+    return proxy(req, resOrSocket, options, onRes, onError)
   } catch (err) {
     return onError(err)
   }