q-koa 13.8.47 → 13.8.49

package/core/app.js

@@ -101,6 +101,26 @@ const connectDatabase = (database) => (config) => {
 const defaultConfig = require('./config')
 
 const is_dev = process.env.NODE_ENV !== 'production'
+
+// JWT 兜底配置（应用未配置 jwt 时使用），提为模块级常量避免每个请求重建对象
+const DEFAULT_JWT = {
+  secret: 'token',
+  tokenKey: 'token',
+  expiresIn: '200000000',
+  _whiteList: [],
+  whiteList: [
+    'find',
+    'administrator/login',
+    'administrator/checkLogin',
+    'user/checkLogin',
+    'user/login',
+    'user/register',
+    'common',
+    'good/search',
+    'category/tree',
+    'system',
+  ],
+}
 class APP {
   constructor(_config) {
     const config = _.defaultsDeep(_config, defaultConfig)
@@ -181,18 +201,13 @@ class APP {
     this.app.use(async (ctx, next) => {
       const { url, host, ip, method } = ctx.request
 
-      if (
-        ctx.request.header['user-agent'] &&
-        ctx.request.header['user-agent'].includes('Tencent Security')
-      ) {
+      const userAgent = ctx.request.header['user-agent'] || ''
+      if (userAgent.includes('Tencent Security')) {
         return ctx.ERROR('error')
       }
 
-      if (
-        ctx.request.header['user-agent'] &&
-        ctx.request.header['user-agent'].includes('Apache-HttpClient')
-      ) {
-        throw new Error(`error`)
+      if (userAgent.includes('Apache-HttpClient')) {
+        throw new Error('error')
       }
 
       if (this.config.blackList.includes(ip)) {
@@ -208,24 +223,7 @@ class APP {
       const { secret, whiteList, _whiteList, tokenKey, expiresIn } =
         ctx.app[appName] && ctx.app[appName].appConfig
           ? ctx.app[appName].appConfig.jwt
-          : {
-              secret: 'token',
-              tokenKey: 'token',
-              expiresIn: '200000000',
-              _whiteList: [],
-              whiteList: [
-                'find',
-                'administrator/login',
-                'administrator/checkLogin',
-                'user/checkLogin',
-                'user/login',
-                'user/register',
-                'common',
-                'good/search',
-                'category/tree',
-                'system',
-              ],
-            }
+          : DEFAULT_JWT
 
       /**
        *  挂在jwt解密方法
@@ -296,7 +294,12 @@ class APP {
 
     this.app.on('error', async (err, ctx) => {
       // AggregateError（Node 17+ DNS 双栈解析等场景）取第一个子错误作为实际错误
-      if (err && err.name === 'AggregateError' && Array.isArray(err.errors) && err.errors.length > 0) {
+      if (
+        err &&
+        err.name === 'AggregateError' &&
+        Array.isArray(err.errors) &&
+        err.errors.length > 0
+      ) {
         err = err.errors[0]
       }
       const { url } = ctx.request
@@ -333,9 +336,7 @@ class APP {
           ? this.config.includes[0]
           : this.config.includes[0].app
       if (!this.app[appName]) {
-        throw new Error(
-          `没有生成应用${this.config.includes[0]}，请检查config includes`
-        )
+        throw new Error(`没有生成应用${appName}，请检查config includes`)
       }
       this.server = this.app.listen(this.port, () => {
         console.log(
@@ -444,9 +445,7 @@ class APP {
     dirList = dirList.filter((item) => includeList.some((i) => i.app === item))
 
     let dbFlag = false
-    for (let i = 0; i < dirList.length; i++) {
-      const appName = dirList[i]
-
+    for (const appName of dirList) {
       console.log(
         chalk.green(`==================== 启动 ${appName} ====================`)
       )
@@ -671,8 +670,7 @@ class APP {
       (item) => !excludesModel.includes(item)
     )
 
-    for (let i = 0; i < pluginDirResult.length; i++) {
-      const folder = pluginDirResult[i]
+    for (const folder of pluginDirResult) {
       const isFolder = fs
         .lstatSync(path.resolve(pluginDir, folder))
         .isDirectory()
@@ -775,8 +773,7 @@ class APP {
                   hookList = [...hookList, 'beforeBulkDestroy']
                 }
                 let reduceHook = {}
-                for (let i = 0; i < hookList.length; i++) {
-                  const hook = hookList[i]
+                for (const hook of hookList) {
                   reduceHook = {
                     ...reduceHook,
                     [hook]: (target, t) => {
@@ -979,8 +976,7 @@ class APP {
       },
       this.app[appName]
     )
-    for (let i = 0; i < Object.keys(this.app[appName].model).length; i++) {
-      const model = Object.keys(this.app[appName].model)[i]
+    for (const model of modelList) {
 
       // if (!this.app[appName].include || !this.app[appName].include[model]) {
       //   this.app[appName] = _.defaultsDeep(
@@ -1161,8 +1157,8 @@ class APP {
           'controller',
           'afterExecute',
         ]
-        for (let i = 0; i < hooksList.length; i++) {
-          if (_.get(app, `${appName}.${hooksList[i]}.${controller}.${fn}`)) {
+        for (const hook of hooksList) {
+          if (_.get(app, `${appName}.${hook}.${controller}.${fn}`)) {
             // if (hooksList[i] === 'controller') {
             //   process.once('unhandledRejection', (reason, promise) => {
             //     if (
@@ -1181,7 +1177,7 @@ class APP {
             //     }
             //   })
             // }
-            await app[appName][hooksList[i]][controller][fn](ctx)
+            await app[appName][hook][controller][fn](ctx)
           }
         }
 
@@ -1209,9 +1205,9 @@ class APP {
         }
 
         const hooksList = ['beforeValidate', 'validate', 'afterValidate']
-        for (let i = 0; i < hooksList.length; i++) {
-          if (_.get(app, `${appName}.${hooksList[i]}.${controller}.${fn}`)) {
-            await app[appName][hooksList[i]][controller][fn](ctx)
+        for (const hook of hooksList) {
+          if (_.get(app, `${appName}.${hook}.${controller}.${fn}`)) {
+            await app[appName][hook][controller][fn](ctx)
           }
         }
         if (
@@ -2270,7 +2266,12 @@ class APP {
             })
             .catch((e) => {
               // AggregateError 取第一个子错误作为实际错误
-              if (e && e.name === 'AggregateError' && Array.isArray(e.errors) && e.errors.length > 0) {
+              if (
+                e &&
+                e.name === 'AggregateError' &&
+                Array.isArray(e.errors) &&
+                e.errors.length > 0
+              ) {
                 e = e.errors[0]
               }
               console.log('错误', e)

package/core/file/plugins/common/controller.js

@@ -232,87 +232,6 @@ exports.channel = async (ctx) => {
   }
 }
 
-exports.swagger = async (ctx) => {
-  const { app, appName } = getAppByCtx(ctx)
-  if (app.cache && app.cache.get('swagger'))
-    return ctx.SUCCESS(app.cache.get('swagger'))
-
-  const filePath = path.resolve(
-    __dirname,
-    `${process.cwd()}/app/${appName}/plugins`
-  )
-  const apis = []
-  const pathList = await fsPromise.readdir(filePath)
-  for (let i = 0; i < pathList.length; i++) {
-    const folder = pathList[i]
-    const isFolder = fs.lstatSync(path.resolve(filePath, folder)).isDirectory()
-    if (isFolder) {
-      const models = []
-      if (app.model[folder]) {
-        const result = await app.sequelize
-          .getQueryInterface()
-          .describeTable(folder)
-        Object.keys(result).forEach((attr) => {
-          models.push(lodash.merge(app.attributes[folder][attr], result[attr]))
-          // models[attr] = lodash.merge(app.attributes[folder][attr], result[attr]);
-        })
-      }
-
-      const controllers = []
-      const controllerFilePath = path.resolve(filePath, folder, 'controller.js')
-      const isControllerExist = fs.existsSync(controllerFilePath)
-      if (isControllerExist) {
-        const controllerFile = await fsPromise.readFile(
-          controllerFilePath,
-          'utf-8'
-        )
-        const annotations = controllerFile.match(annotationsReg)
-
-        if (annotations) {
-          for (let j = 0; j < annotations.length; j++) {
-            const annotation = annotations[j]
-            let res
-            const obj = {}
-            while ((res = apiReg.exec(annotation))) {
-              res = res.map((item) => item.toLowerCase())
-              if (res[1] === 'param') {
-                const arr = res[3].split('/')
-                if (obj[res[1]]) {
-                  obj[res[1]].push({
-                    key: arr[0],
-                    type: arr[1],
-                    name: arr[2],
-                  })
-                } else {
-                  obj[res[1]] = []
-
-                  obj[res[1]].push({
-                    key: arr[0],
-                    type: arr[1],
-                    name: arr[2],
-                  })
-                }
-              } else {
-                obj[res[1]] = res[3]
-              }
-            }
-            controllers.push(obj)
-          }
-        }
-      }
-      apis.push({
-        name: folder,
-        desc: lodash.get(app, `config.${folder}.name`, `${folder}`),
-        models,
-        controllers,
-      })
-    }
-  }
-  app.cache && app.cache.set('swagger', apis)
-  // console.log(dir)
-  ctx.SUCCESS(apis)
-}
-
 exports.getLocation = async (ctx, _data) => {
   const { app } = getAppByCtx(ctx)
 

package/core/file/plugins/system/controller.js

@@ -215,22 +215,6 @@ exports.initModel = async (ctx) => {
     return ctx.ERROR(`app.model没有${model}`)
   }
 }
-
-exports.customRouter = async (ctx) => {
-  // const {
-  //   app,
-  //   appName,
-  // } = getAppByCtx(ctx);
-
-  // const list = [{
-  //   path: 'jiangong_statis',
-  //   name: '数据面板'
-  // }]
-  const list = []
-  ctx.SUCCESS(list)
-  return list
-}
-
 exports.showTables = async (ctx, _data) => {
   const { app, appName } = getAppByCtx(ctx)
   const { omit = [] } = _data || ctx.request.body
@@ -666,21 +650,6 @@ exports.chat = async (ctx) => {
   }
 }
 
-exports.dashboard = async (ctx) => {
-  // const {
-  //   app,
-  // } = getAppByCtx(ctx);
-
-  const result = [
-    //   {
-    //   name: '用户',
-    //   number: 5,
-    //   route: 'page/user'
-    // }
-  ]
-  ctx.SUCCESS(result)
-}
-
 exports.showModelCount = async (ctx) => {
   const { app, appName } = getAppByCtx(ctx)
   const { is_count = false } = ctx.request.body

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "q-koa",
-  "version": "13.8.47",
+  "version": "13.8.49",
   "description": "",
   "main": "index.js",
   "scripts": {