11

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

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

执行: SELECT * FROM UsersWHERE Usersid=1 限制 1;

一遍一遍又一遍!

获取 / 200 12ms - 780

执行: SELECT * FROM UsersWHERE Usersid=1 限制 1;

执行: SELECT * FROM UsersWHERE Usersid=1 限制 1;

一遍一遍又一遍!

获取 /js/ui.js 304 4ms

一遍一遍又一遍!

获取 /stylesheets/main.css 304 6ms

执行: SELECT * FROM UsersWHERE Usersid=1 限制 1;

一遍一遍又一遍!

获取 /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 文件运行?如果是这样,有没有办法减少对数据库的请求数量?

4

1 回答 1

23

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

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

于 2013-03-05T07:50:06.480 回答