From c32e607578b99ad4b3e7cf621b5327e2831643f4 Mon Sep 17 00:00:00 2001 From: tkvern Date: Tue, 13 Oct 2020 00:00:09 +0800 Subject: [PATCH 1/2] fix: when middleware config options is undefined. options.enable expression will be error occurred --- packages/web/app.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/web/app.js b/packages/web/app.js index f00fe09a8fc1..bd20e597bd24 100644 --- a/packages/web/app.js +++ b/packages/web/app.js @@ -39,7 +39,7 @@ class AppBootHook { } else { // egg const options = this.app.config[name]; - if (options.enable === false) { + if (options && options.enable === false) { continue; } // support options.match and options.ignore From a088e13c1dbbb62a88efcd1d81a9d9193567a523 Mon Sep 17 00:00:00 2001 From: tkvern Date: Tue, 13 Oct 2020 09:33:53 +0800 Subject: [PATCH 2/2] fix: when middleware config options is undefined, options.match and options.ignore unable to work --- packages/web/app.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/web/app.js b/packages/web/app.js index bd20e597bd24..cf4e166ef5ac 100644 --- a/packages/web/app.js +++ b/packages/web/app.js @@ -38,8 +38,8 @@ class AppBootHook { this.app.use(mwIns); } else { // egg - const options = this.app.config[name]; - if (options && options.enable === false) { + const options = this.app.config[name] || {}; + if (options.enable === false) { continue; } // support options.match and options.ignore