路由鉴权中考虑await，而不是一个同步方法

[MrTreasure]

有注意到你在 engine
中是使用了 Reflect.getMetadata, 但是我的TS却显示Reflect上没有这个方法，请问应该怎么设置呢？

[MrTreasure]

reflect-metadata 这个库，用来在类的原型上或者方法上定义元数据

[yvanwangl]

@MrTreasure 你在项目中安装一下 'reflect-metadata' ， 然后在入口文件中 import 'reflect-metadata';

[MrTreasure]

                    let controllerAuth = Reflect.getMetadata(rootAuthSymbolKey, target);
                    if (controllerAuth && typeof controllerAuth == 'function') {
                        // 这里应该添加 await
                        let hasAuth = controllerAuth(ctx, next);
                        if (!hasAuth) {
                            ctx.response.status = 403;
                            ctx.response.body = 'Permission Denied!';
                            return null;
                        }
                    }
                    if (authFunc && typeof authFunc == 'function') {
                        let hasAuth = authFunc(ctx, next);
                        if (!hasAuth) {
                            ctx.response.status = 403;
                            ctx.response.body = 'Permission Denied!';
                            return null;
                        }
                    }

这里是不是添加为一个Promise好一点，因为一般鉴权都是从redis或者是数据库读取消息，必然是一个异步的方法

[yvanwangl]

@MrTreasure 嗯，这是个不错的建议，下一个版本会加入 ：）

[MrTreasure]

注意到你在执行注册路由方法的时候传递了一个instance 参数进去，想作为后面的this

let instance = new IwinterController()

self.router[httpMethod](`${self.prefix}${rootPath}${path}`, method(instance));

decorator.value = (instance) => async (ctx, next) => {}

let result =await oldMethod.apply(instance, params);

在第三个步骤中，可以不使用箭头函数
而写为

decorator.value = async function() {
  let result = await oldMethod.apply(this, arguments)
}

这样避免了instance的传递，方便理解。这里的this就是指向实例

在 AOP 中

if(beforeHook && typeof beforeHook == 'function' ) {
     beforeHook(ctx, next);
}

这里是否可以考虑加一个标记表明 切面是否是异步函数，需要做异步处理。虽然一个切面不应该对原函数进行影响，但是可能某些情况下需要这样的修改，可以添加一个异步函数的处理方法，但默认为同步函数