fix: SimpleAuthentication should work with custom basePaths

Unleash · Mar 9, 2019 · e0eac6c · e0eac6c
1 parent 7e0913b
commit e0eac6c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/lib/app.js b/lib/app.js
@@ -39,7 +39,7 @@ module.exports = function(config) {
     }
 
     if (config.adminAuthentication === 'unsecure') {
-        simpleAuthentication(app);
+        simpleAuthentication(baseUriPath, app);
     }
 
     if (typeof config.preRouterHook === 'function') {

diff --git a/lib/middleware/simple-authentication.js b/lib/middleware/simple-authentication.js
@@ -3,31 +3,31 @@
 const User = require('../user');
 const AuthenticationRequired = require('../authentication-required');
 
-function unsecureAuthentication(app) {
-    app.post('/api/admin/login', (req, res) => {
+function unsecureAuthentication(basePath = '', app) {
+    app.post(`${basePath}/api/admin/login`, (req, res) => {
         const user = req.body;
         req.session.user = new User({ email: user.email });
         res.status(200)
             .json(req.session.user)
             .end();
     });
 
-    app.use('/api/admin/', (req, res, next) => {
+    app.use(`${basePath}/api/admin/`, (req, res, next) => {
         if (req.session.user && req.session.user.email) {
             req.user = req.session.user;
         }
         next();
     });
 
-    app.use('/api/admin/', (req, res, next) => {
+    app.use(`${basePath}/api/admin/`, (req, res, next) => {
         if (req.user) {
             next();
         } else {
             return res
                 .status('401')
                 .json(
                     new AuthenticationRequired({
-                        path: '/api/admin/login',
+                        path: `${basePath}/api/admin/login`,
                         type: 'unsecure',
                         message:
                             'You have to indentify yourself in order to use Unleash.',