feat(proxy): set default value of proxy.changeOrigin to true (#66)

svrxjs · Aug 26, 2019 · 627db59 · 627db59
1 parent 0999450
commit 627db59
Show file tree

Hide file tree

Showing 5 changed files with 643 additions and 6,092 deletions.
diff --git a/examples/proxy/.svrxrc.js b/examples/proxy/.svrxrc.js
@@ -6,7 +6,6 @@ module.exports = {
     {
       context: ['/api', '/same'],
       target: 'https://randomuser.me',
-      changeOrigin: true,
       pathRewrite: {
         '/same/api': '/api'
       }

diff --git a/packages/svrx-util/package-lock.json b/packages/svrx-util/package-lock.json
diff --git a/packages/svrx/__tests__/spec/svrx.plugin.proxy.js b/packages/svrx/__tests__/spec/svrx.plugin.proxy.js
@@ -14,10 +14,10 @@ describe('Proxy Action', async () => {
             const { route } = router;
             route(({ get }) => {
               get('/api(.*)').to.proxy(PROXY_SERVER);
-              get('/origin/api/test').to.proxy(PROXY_SERVER, {
-                changeOrigin: true,
+              get('/origin/api/test').to.proxy(PROXY_SERVER);
+              get('/origin/api/noset').to.proxy(PROXY_SERVER, {
+                changeOrigin: false,
               });
-              get('/origin/api/noset').to.proxy(PROXY_SERVER);
               get('/rewrite/api(.*)').to.proxy(PROXY_SERVER, {
                 pathRewrite: {
                   '^/rewrite/api': '/api',

diff --git a/packages/svrx/lib/plugin/svrx-plugin-proxy/index.js b/packages/svrx/lib/plugin/svrx-plugin-proxy/index.js
@@ -50,7 +50,7 @@ function hasPort(host) {
 
 async function proxy({ proxyRule, ctx }) {
   const {
-    target, pathRewrite, changeOrigin, secure = true,
+    target, pathRewrite, changeOrigin = true, secure = true,
   } = proxyRule;
   const path = rewritePath(ctx.originalUrl, pathRewrite);
   const urlObj = new libUrl.URL(path, target);