From 7732d52583f7c0f4afbff3c90148bc0ff70a7ad2 Mon Sep 17 00:00:00 2001 From: Nick O'Leary Date: Wed, 12 Jan 2022 13:26:18 +0000 Subject: [PATCH] Initialize passport when only adminAuth.tokens is set Fixes #3341 --- .../node_modules/@node-red/editor-api/lib/auth/strategies.js | 3 ++- packages/node_modules/@node-red/editor-api/lib/index.js | 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/node_modules/@node-red/editor-api/lib/auth/strategies.js b/packages/node_modules/@node-red/editor-api/lib/auth/strategies.js index 7a77354fa..a6bc8e94e 100644 --- a/packages/node_modules/@node-red/editor-api/lib/auth/strategies.js +++ b/packages/node_modules/@node-red/editor-api/lib/auth/strategies.js @@ -146,7 +146,7 @@ function authenticateUserToken(req) { } else { reject(); } - }); + }).catch(reject); } else { reject(); } @@ -163,6 +163,7 @@ TokensStrategy.prototype.authenticate = function(req) { authenticateUserToken(req).then(user => { this.success(user,{scope:user.permissions}); }).catch(err => { + log.trace("token authentication failure: "+err.stack) this.fail(401); }); } diff --git a/packages/node_modules/@node-red/editor-api/lib/index.js b/packages/node_modules/@node-red/editor-api/lib/index.js index 258e6e514..56f52a222 100644 --- a/packages/node_modules/@node-red/editor-api/lib/index.js +++ b/packages/node_modules/@node-red/editor-api/lib/index.js @@ -90,6 +90,8 @@ function init(settings,_server,storage,runtimeAPI) { auth.getToken, auth.errorHandler ); + } else if (settings.adminAuth.tokens) { + adminApp.use(passport.initialize()); } adminApp.post("/auth/revoke",auth.needsPermission(""),auth.revoke,apiUtil.errorHandler); }