为什么res.send被调用两次?

时间:2017-01-05 16:05:56

标签: node.js express

我正致力于为快速路由器创建一个中间件,该路由器将为每个请求和响应执行一些代码。拦截请求很容易,并且有很多例子,但我没有找到任何优雅的方法来拦截响应。经过一些研究,我提出的最好的方法是替换响应对象的send函数,如下面的代码片段所示:

const express = require('express');
const app = express();
var router = express.Router();
var port = process.env.PORT || 3000;

router.get('/test', function(req, res) {
    res.send({
        message: "testing"
    });
});

app.use(function(req, res, next){
    console.log("INTERCEPT-REQUEST");
    const orig_send = res.send;
    res.send = function(arg) {
        console.log("INTERCEPT-RESPONSE");
        orig_send.call(res, arg);
    };
    next();
});

app.use("/api", router);
app.listen(process.env.PORT || 3000) && console.log("Running");

这种方法存在问题:出于某种原因" INTERCEPT-RESPONSE"在控制台中打印两次,这意味着res.send被调用两次......

我可以在第一次调用res.locals时设置一个标志,以避免两次处理响应,但我想知道为什么res.send被调用两次?

1 个答案:

答案 0 :(得分:8)

更好的例子

尝试使用此代码查看传递给res.send的参数:

const express = require('express');
const app = express();
var router = express.Router();
var port = process.env.PORT || 3000;

router.get('/test', function(req, res) {
    console.log('ACTUAL RESPONSE');
    res.send({
        message: "testing"
    });
});

app.use(function(req, res, next){
    console.log("INTERCEPT-REQUEST");
    const orig_send = res.send;
    res.send = function(arg) {
        console.log("INTERCEPT-RESPONSE", JSON.stringify(arguments));
        orig_send.call(res, arg);
    };
    next();
});

app.use("/api", router);
app.listen(process.env.PORT || 3000, function () {
    console.log("Running");
});

(我还更改了“Running”的打印,以便在服务器实际收听时打印它 - 在服务器监听之前打印了代码&& - 但这并不重要。)

现在运行后:

curl http://localhost:3000/api/test

服务器控制台上的输出是:

Running
INTERCEPT-REQUEST
ACTUAL RESPONSE
INTERCEPT-RESPONSE {"0":{"message":"testing"}}
INTERCEPT-RESPONSE {"0":"{\"message\":\"testing\"}"}

会发生什么

正如您所看到的,您的处理程序实际上是由您的代码调用一次,并将对象作为第一个(也是唯一的)参数。但随后将一个序列化为JSON的对象再次调用。这就是res.send内部工作的方式 - 详见下文。既然你把拦截函数放在了实际的响应对象上,那么我猜它是用JSON参数调用自己的,它甚至不知道它在此期间调用你的函数。

如何避免

尝试使用自己序列化为JSON的对象:

const express = require('express');
const app = express();
var router = express.Router();
var port = process.env.PORT || 3000;

router.get('/test', function(req, res) {
    console.log('ACTUAL RESPONSE');
    res.send(JSON.stringify({
        message: "testing"
    }));
});

app.use(function(req, res, next){
    console.log("INTERCEPT-REQUEST");
    const orig_send = res.send;
    res.send = function(arg) {
        console.log("INTERCEPT-RESPONSE", JSON.stringify(arguments));
        orig_send.call(res, arg);
    };
    next();
});

app.use("/api", router);
app.listen(process.env.PORT || 3000, function () {
    console.log("Running");
});

现在打印:

Running
INTERCEPT-REQUEST
ACTUAL RESPONSE
INTERCEPT-RESPONSE {"0":"{\"message\":\"testing\"}"}

仅调用res.send一次。

说明

现在,这是处理res.json的对象参数的代码:

  if (chunk === null) {
    chunk = '';
  } else if (Buffer.isBuffer(chunk)) {
    if (!this.get('Content-Type')) {
      this.type('bin');
    }
  } else {
    return this.json(chunk);
  }

请参阅:https://github.com/expressjs/express/blob/master/lib/response.js#L144-L154

你得到了else分支,它会用你的论点调用this.json()(真的是res.json())。

但请注意 - res.json()在此行中调用res.send()

return this.send(body);

请参阅:https://github.com/expressjs/express/blob/master/lib/response.js#L250

在运行真实res.send()之前,会调用您的拦截功能(第二次)。

所以,神秘解决了。 :)