From aa75d50361d7705e030be7725464b472c0e7f9cc Mon Sep 17 00:00:00 2001 From: Dinawo Date: Sat, 30 Mar 2024 16:13:56 +0100 Subject: [PATCH] Refactor ActiveDirectory authentication strategy --- models/Passport-ActiveDirectory.js | 2 +- routes/auth.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/models/Passport-ActiveDirectory.js b/models/Passport-ActiveDirectory.js index e7c246d..4586925 100644 --- a/models/Passport-ActiveDirectory.js +++ b/models/Passport-ActiveDirectory.js @@ -7,7 +7,7 @@ const { getUserData } = require('../Middlewares/watcherMiddleware'); const setupFilePath = path.join('setup.json'); const setupData = JSON.parse(fs.readFileSync(setupFilePath, 'utf-8')); -passport.use(new ActiveDirectoryStrategy({ +passport.use('ActiveDirectory', new ActiveDirectoryStrategy({ integrated: false, ldap: { url: setupData.ldap.url, diff --git a/routes/auth.js b/routes/auth.js index 33a404c..96e6669 100644 --- a/routes/auth.js +++ b/routes/auth.js @@ -53,7 +53,7 @@ router.get('/logout', (req, res) => { var opts = { failWithError: true } router.post('/activedirectory', (req, res, next) => { - passport.authenticate('ActiveDirectory', opts, (err, user, info) => { + passport.authenticate('ActiveDirectory', (err, user) => { if (err) { return res.render('AuthLogin', { isAuthenticated: false, errorMessage: err.message, setupData: {}, showActiveDirectoryForm: true, currentUrl: req.originalUrl }); }