Почему ACL(nodejs) блокирует отображение контента?

Question

[Адель Халитов]

Всем привет, имею свой проект на angular - frontend, nodejs(express, passport, acl) - backend.

config.js

mongoose.connect(myURLtoDB, {useCreateIndex: true, useNewUrlParser: true}, function (err, db) {
    .....
};
var conn    = mongoose.connection;
exports.freshConnect = conn;

router.js

var db = require('../config/config.js');
var node_acl = require('acl');
var acl = new node_acl( new node_acl.mongodbBackend(db.freshConnect, 'acl_') );

router.use('/crm', checkForPermissions(), crmRouter);

function checkForPermissions() {
    console.log('Я работаю')
    return acl.middleware(2, getUserId);
}

function getUserId(req) {
    if (req.user) {
        return req.session.passport.user;
    }
}

Теперь при запуске базы данных в консоле сразу же запускается checkForPermissions(), хотя я никакие запросы не делаю.

При попытке отобразить какой то контнт в angular, например по URL: crm/newlead, контент не отображается. С чем это может быть связанно? Что я делаю не так?

У меня на каждой странице /crm/* идет на сервер запрос passport.js isAuthenticated(), который возвращает true или false (Если true значит пользователь авторизирован и имеет доступ к этой странице, если false идет редирект на страницу логина), но переходя на страницу angular отправляет запрос на сервер, для того чтобы получить ответ функции isAuthenticated(), но ответа не получает, тк запроса до сервера не доходит и функция checkForPermissions() вызывается в момент запуска сервера, при последующих запросах они ничего не выводит. С чем это связанно?

Answer 1

[Антон Швец]

попробуйте

var db = require('../config/config.js');
var node_acl = require('acl');
var acl = new node_acl( new node_acl.mongodbBackend(db.freshConnect, 'acl_') );

router.use('/crm', checkForPermissions, crmRouter);

function checkForPermissions(req, res, next) {
    console.log('Я работаю')
    const userId = getUserId(req);
    acl.middleware(2, userId)(req, res, next);
}

function getUserId(req) {
    if (req.user) {
        return req.session.passport.user;
    }
}

а почему у вас getUserId(req) такая странная? так точно должно быть?

Comments

[Адель Халитов]

Картина не изменилась( Все в точности так же!

а почему у вас getUserId(req) такая странная? так точно должно быть?

Работает верно, она возвращает id пользователя из сессии

[Антон Швец]

Адель Халитов, похоже ошибка не в этом месте. Тут я не вижу сходу никаких проблем.

Все в точности так же!

Должно быть отличие. У вас "Я работаю" должно выводится при монтировании роутера, у меня - при запросе.

[Адель Халитов]

Антон Швец, Имеется в виду что router.use('/crm', checkForPermissions, crmRouter); сheckForPermissions должна быть без ()?
"Я работаю" срабатывает при запросе, не при монтировании(исправлено, после указании ошибки Stockholm Syndrome ).

router.use('/crm', crmRouter);

Отправляю запрос - auth.guard.ts:21 
Получаю ответ - auth.guard.ts:25

router.use('/crm', checkForPermissions, crmRouter);
Отправляю запрос - auth.guard.ts:21

Это запрос который должен получить сервер:

configRouter.get('/detect', function( req, res) {
    console.log("Запрос получил")
    if ( req.isAuthenticated() ) {
        res.status(200).send({ detect: true })
    } else {
        res.status(401).send({ detect: false })
    }
})

console.log("Запрос получил") - в консоли не отображается, следсвие он не получает этого запроса, получается мидлвейр блокирует его

[Антон Швец]

Адель Халитов, у вас не было ошибки, вы просто console.log не туда поместили.
acl.middleware() возвращает express.RequestHandler, т.е. функцию-миддлварь, которую вы должны передать роутеру.

А если вы в своем примере в роутер поставите просто checkForPermissions, как вам "посоветовали", то запрос зависнет после вызова console.log, потому что в качестве миддлвари вы передали роутеру саму checkForPermissions, и экспресс будет ждать вызова next, которого у вас вообще нет.

[Адель Халитов]

Антон Швец, Немножко не понял, не сложно привести пример?
Я делал в точности как в вашем примере

[Антон Швец]

Адель Халитов, аааа, куда уж проще то

function getMiddleware() { // аналогична acl.middleware()
  return (req, res, next) => {
     console.log('getMiddleware works');
     next();
  };
}
/** должна быть использована так **/
router.use('*', getMiddleware(), doSomething);

/** а "посоветовали" сделать так **/
router.use('*', getMiddleware, doSomething);
/** в результате выведет getMiddleware works и запрос навечно повиснет **/

читайте expressjs.com/ru/guide/using-middleware.html и помните, что funcName() даст вам результат вызова функции в этом месте. getMiddleware при вызове возвращает функцию.

[Адель Халитов]

Антон Швец, Идеально! Спасибо огромное!

router.use('/crm', checkForPermissions(), crmRouter);

function checkForPermissions() {
    return (req, res, next) => {
        console.log('Я работаю')
        const userId = getUserId(req);
        db.acl.middleware(2, userId);
        next();
    }
   
}

function getUserId(req) {
    if (req.user) {
        console.log(req.session.passport.user)
        return req.session.passport.user;
    }
}