@abtnode/router-provider 1.16.13-beta-d13e9789 → 1.16.14-beta-a898bfcb

package/lib/nginx/index.js

@@ -9,13 +9,11 @@ const kill = require('fkill');
 const getPort = require('get-port');
 const uniqBy = require('lodash/uniqBy');
 const isEmpty = require('lodash/isEmpty');
-const cloneDeep = require('lodash/cloneDeep');
 const formatBackSlash = require('@abtnode/util/lib/format-back-slash');
 const {
   DOMAIN_FOR_DEFAULT_SITE,
   ROUTING_RULE_TYPES,
   CONFIG_FOLDER_NAME,
-  DEFAULT_ADMIN_PATH,
   SLOT_FOR_IP_DNS_SITE,
   WELLKNOWN_SERVICE_PATH_PREFIX,
   USER_AVATAR_PATH_PREFIX,
@@ -57,9 +55,6 @@ const {
 
 const REQUIRED_MODULES = [{ configName: 'with-stream', moduleBinaryName: 'ngx_stream_module.so' }];
 
-// TODO: move the did-auth-path-prefix to a constant
-const ADMIN_DID_AUTH_PATH_PREFIX = `${DEFAULT_ADMIN_PATH}/api/did`;
-
 // convert wildcard domain and ipDnsDomain template to regex
 const parseServerName = (domain) => {
   // ipDnsDomain template
@@ -136,10 +131,6 @@ class NginxProvider extends BaseProvider {
     this.initialize();
   }
 
-  static isDidAuthPrefix(prefix) {
-    return prefix === ADMIN_DID_AUTH_PATH_PREFIX;
-  }
-
   getRelativeConfigDir(dir) {
     return path.relative(this.configDir, dir);
   }
@@ -190,14 +181,7 @@ class NginxProvider extends BaseProvider {
         conf.on('flushed', () => resolve());
         conf.live(this.configPath);
 
-        const { sites, configs: siteCorsConfigs } = formatRoutingTable(routingTable, (rules, rule) => {
-          // TODO: this is really hacky, and should be abstracted out
-          if (rule.type === ROUTING_RULE_TYPES.DAEMON && rule.prefix === DEFAULT_ADMIN_PATH) {
-            const clonedRule = cloneDeep(rule);
-            clonedRule.prefix = ADMIN_DID_AUTH_PATH_PREFIX;
-            rules.push(clonedRule);
-          }
-        });
+        const { sites, configs: siteCorsConfigs } = formatRoutingTable(routingTable);
 
         this._addCorsMap(conf, siteCorsConfigs);
         conf.nginx.http._add('server_tokens', 'off');
@@ -475,10 +459,9 @@ class NginxProvider extends BaseProvider {
     server._add('location', concatPath(prefix, suffix));
 
     const location = this._getLastLocation(server);
-    const isDidAuthPath = NginxProvider.isDidAuthPrefix(prefix);
 
     // Note: 下面这段代码比较 tricky，不要在这段代码之前添加任何 add_header, proxy_set_header, proxy_hide_header 的语句，否则 nginx 配置可能无法按预期工作
-    this._addCors({ location, corsAllowedOrigins, serverName, isDidAuthPath });
+    this._addCors({ location, corsAllowedOrigins, serverName });
 
     this._addCommonResHeaders(location, commonHeaders);
     if (!cacheGroup && !suffix) {
@@ -539,10 +522,9 @@ class NginxProvider extends BaseProvider {
     }
 
     // Redirect daemon traffic
-    const rewritePathPrefix = isDidAuthPath ? DEFAULT_ADMIN_PATH : prefix;
-    location._add('proxy_set_header', `X-Path-Prefix "${rewritePathPrefix}"`);
+    location._add('proxy_set_header', `X-Path-Prefix "${prefix}"`);
     if (!suffix && prefix !== target) {
-      location._add('rewrite', `^${rewritePathPrefix}/?(.*) ${`${target}/`.replace(/\/\//g, '/')}$1  break`);
+      location._add('rewrite', `^${prefix}/?(.*) ${`${target}/`.replace(/\/\//g, '/')}$1  break`);
     }
 
     location._add('proxy_pass', `http://${getUpstreamName(port)}`);
@@ -553,9 +535,7 @@ class NginxProvider extends BaseProvider {
     server._add('location', `${concatPath(prefix, suffix)}`);
     const location = this._getLastLocation(server);
 
-    const isDidAuthPath = NginxProvider.isDidAuthPrefix(prefix);
-
-    this._addCors({ location, corsAllowedOrigins, serverName, isDidAuthPath });
+    this._addCors({ location, corsAllowedOrigins, serverName });
 
     location._add('set $abt_query_string', '""');
     location._addVerbatimBlock('if ($query_string)', 'set $abt_query_string "?$query_string";');
@@ -897,9 +877,9 @@ class NginxProvider extends BaseProvider {
     });
   }
 
-  _addCors({ location, corsAllowedOrigins, serverName, isDidAuthPath }) {
+  _addCors({ location, corsAllowedOrigins, serverName }) {
     if (!isEmpty(corsAllowedOrigins)) {
-      if (corsAllowedOrigins.includes(DOMAIN_FOR_DEFAULT_SITE) || isDidAuthPath) {
+      if (corsAllowedOrigins.includes(DOMAIN_FOR_DEFAULT_SITE)) {
         location._add('include', 'includes/cors-loose');
         location._add('include', 'includes/security');
       } else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@abtnode/router-provider",
-  "version": "1.16.13-beta-d13e9789",
+  "version": "1.16.14-beta-a898bfcb",
   "description": "Routing engine implementations for abt node",
   "author": "polunzh <polunzh@gmail.com>",
   "homepage": "https://github.com/ArcBlock/blocklet-server#readme",
@@ -32,10 +32,10 @@
     "url": "https://github.com/ArcBlock/blocklet-server/issues"
   },
   "dependencies": {
-    "@abtnode/constant": "1.16.13-beta-d13e9789",
-    "@abtnode/logger": "1.16.13-beta-d13e9789",
-    "@abtnode/router-templates": "1.16.13-beta-d13e9789",
-    "@abtnode/util": "1.16.13-beta-d13e9789",
+    "@abtnode/constant": "1.16.14-beta-a898bfcb",
+    "@abtnode/logger": "1.16.14-beta-a898bfcb",
+    "@abtnode/router-templates": "1.16.14-beta-a898bfcb",
+    "@abtnode/util": "1.16.14-beta-a898bfcb",
     "@arcblock/http-proxy": "^1.19.1",
     "axios": "^0.27.2",
     "debug": "^4.3.4",
@@ -59,5 +59,5 @@
     "bluebird": "^3.7.2",
     "fs-extra": "^10.1.0"
   },
-  "gitHead": "fcb07d0b3c742aaa68101e302b9eca193d0db872"
+  "gitHead": "1e6f2ec5199ca28e106a6b02114f2a3f355b3ceb"
 }