forward request body & method

billiegoose · billiegoose · commit 19f957a6073a · 2017-08-04T00:40:30.000-04:00
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -0,0 +1,17 @@
+{
+    // Use IntelliSense to learn about possible Node.js debug attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "type": "node",
+            "request": "launch",
+            "name": "Launch micro",
+            "program": "${workspaceRoot}/node_modules/micro/bin/micro.js",
+            "args": [
+                "index.js"
+            ]
+        }
+    ]
+}
diff --git a/README.md b/README.md
@@ -34,25 +34,33 @@ JavaScript, there is no way we could be impersonating Alice. Alice is safe.
 Bob was never protected in the first place. Eve has better things to do with
 her time.
 
+## But I need to POST/PUT/etc with data?
+
+That works too! Just make an OPTIONS/POST/PUT/DELETE/etc request and it will be forwarded.
+If you can only make GET requests, you can provide a `method` query parameter and
+the server will make it that kind of request instead.
+
 ## Supported headers
 
 If there's a way to whitelist ALL headers, let me know. The one's I've explicitly added
 so far are:
 
-- x-requested-with
+- accept-encoding
+- accept-language
+- accept
 - access-control-allow-origin
-- x-http-method-override
-- content-type
 - authorization
-- accept
-- connection
-- pragma
 - cache-control
+- connection
+- content-length
+- content-type
 - dnt
-- referer
-- accept-encoding
-- accept-language
+- pragma
 - range
+- referer
+- user-agent
+- x-http-method-override
+- x-requested-with
 
 ## That is nice, I want to run my own server
 
diff --git a/index.js b/index.js
@@ -3,20 +3,22 @@ const url = require('url')
 const pkg = require('./package.json')
 const {send} = require('micro')
 const allowHeaders = [
-  'x-requested-with',
+  'accept-encoding',
+  'accept-language',
+  'accept',
   'access-control-allow-origin',
-  'x-http-method-override',
-  'content-type',
   'authorization',
-  'accept',
-  'connection',
-  'pragma',
   'cache-control',
+  'connection',
+  'content-length',
+  'content-type',
   'dnt',
-  'referer',
-  'accept-encoding',
-  'accept-language',
+  'pragma',
   'range',
+  'referer',
+  'user-agent',
+  'x-http-method-override',
+  'x-requested-with',
 ]
 const cors = require('micro-cors')({allowHeaders})
 const fetch = require('node-fetch')
@@ -29,8 +31,20 @@ async function service (req, res) {
     <html>
       <title>400 Error</title>
       <h1>Missing 'href' parameter.</h1>
-      <h1>Note: An optional 'method' parameter is also supported.</h1>
       <h2>See docs: <a href="https://npmjs.org/package/${pkg.name}">https://npmjs.org/package/${pkg.name}</a></h2>
+      <h2>Supported HTTP Methods</h2>
+      <ul>
+        <li>All - OPTIONS, GET, POST, PUT, DELETE, etc</li>
+      </ul>
+      <h2>Supported Query Parameters</h2>
+      <ul>
+        <li>?href=&lt;href&gt; - <i>required</i>, the URL you are trying to reach</li>
+        <li>&method=&lt;method&gt; - <i>optional</i>, the HTTP method to use</li>
+      </ul>
+      <h2>Supported Headers</h2>
+      <ul>
+        ${allowHeaders.map(x => `<li>${x}</li>`).join('\n')}
+      </ul>
     </html>
     `
     return send(res, 400, html)
@@ -42,8 +56,9 @@ async function service (req, res) {
     }
   }
   let f = await fetch(q.href, {
-    method: q.method || 'GET',
-    headers
+    method: q.method || req.method,
+    headers,
+    body: req
   })
   f.body.pipe(res)
 }
