【问题标题】:passport.deserializeUser executing a DB (sequelize) command for each HTTP requestpassport.deserializeUser 为每个 HTTP 请求执行 DB(续集)命令
【发布时间】:2013-02-19 05:52:56
【问题描述】:

我使用 sequelize 作为 ORM 和 passport.js (passport-local) 进行身份验证。我注意到每个 HTTP 请求都会产生一个单独的数据库命令。我开始研究 deserializeUser() 函数。

加载单个页面时,这是我得到的:

执行:SELECT * FROM Users WHERE Users.id=1 LIMIT 1;

一遍又一遍!

GET / 200 12ms - 780

执行:SELECT * FROM Users WHERE Users.id=1 LIMIT 1;

执行:SELECT * FROM Users WHERE Users.id=1 LIMIT 1;

一遍又一遍!

GET /js/ui.js 304 4ms

一遍又一遍!

GET /stylesheets/main.css 304 6ms

执行:SELECT * FROM Users WHERE Users.id=1 LIMIT 1;

一遍又一遍!

GET /images/logo.jpg 304 3ms

这是 passport.deserializeUser 的外观:

passport.deserializeUser(function(id, done) {
    User.find(id).success(function(user) {
        console.log('Over and over and over!');
        done(null, user);
    }).error(function(err) {
        done(err, null);
    });
});

我请求的页面是:

index: function(req, res) {
    res.render('index', {
        title: "Welcome to EKIPLE!",
        currentUser: req.user
    });
}

deserializeUser 是否应该为请求的每个图像、html、css 文件运行?如果是这样,有没有办法减少对数据库的请求数量?

【问题讨论】:

    标签: node.js sequelize.js passport.js


    【解决方案1】:

    这是中间件顺序不正确的典型结果。你应该app.use(或同等的)处理静态资源的中间件(通常是express.staticconnect.static你之前你app.use Passport 中间件。其他处理不需要通过 Passport 运行的请求的中间件也是如此。

    这样,对静态资源的请求将永远不会到达 Passport 中间件,因此不会导致那些不必要的数据库请求。

    【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多