Node.js发送POST请求时出现[Error: socket hang up] code: 'ECONNRESET'错误

Question

我创建了一个示例来将数据发布到REST服务，发现当我有非ASCII或非拉丁字符（请参见data.firstName）时，使用TEST-REST.js的POST请求会引发错误：

错误：{[Error: socket hang up] code：'ECONNRESET' }。

// TEST-REST.js
var http = require('http');

var data = JSON.stringify({
  firstName: 'JoaquÌn',
});

var options = {
  host: '127.0.0.1',
  port: 3000,
  path: '/users',
  method: 'POST',
  headers: {
    'Content-Type': 'application/json',
    'Content-Length': data.length
  }
};

var req = http.request(options, function(res) {
  var result = '';

  res.on('data', function(chunk) {
    result += chunk;
  });

  res.on('end', function() {
    console.log(result);
  });
});

req.on('error', function(err) {
  console.log(err);
});

req.write(data);
req.end();

在我的REST服务中，它抛出了这样的错误：

SyntaxError: Unexpected end of input Sun Sep 08 2013 23:25:02 GMT-0700 (PDT) -     at Object.parse (native)
    at IncomingMessage.<anonymous> (/Volumes/Data/Program_Data/GitHub/app/node_modules/express/node_modules/connect/lib/middleware/json.js:66:27) info    at IncomingMessage.EventEmitter.emit (events.js:92:17)
    at _stream_readable.js:920:16 : - - - [Mon, 09 Sep 2013 06:25:02 GMT] "POST /users HTTP/1.1" 400 - "-" "-"
    at process._tickDomainCallback (node.js:459:13)

如果我把firstName的值从“JoaquÌn”替换为“abc”，一切都可以正常工作。我认为我还缺少一些支持或转义来使它正常工作。

有没有人有任何想法如何解决这个问题？我也尝试了以下内容：require('querystring').escape(model.givenName)，它可以工作，但我并不满意。

更新： 我发现如果我注释掉：app.use(express.bodyParser());，错误就会消失。

- Nam Nguyen

---

尝试使用'Content-Type': 'application/json; charset=utf-8'。 - vinayr

你能否在没有“Content-length”头的情况下进行测试？ - user568109

实际上这是连接问题：https://github.com/visionmedia/express/issues/1749。通过更改 'Content-Length': Buffer.byteLength(data) 解决。 - Nam Nguyen

这是一个非常好的发现，Nam。在你提醒我之前，我并不知道这个漏洞。感谢你发布这个问题！ - Eric Elliott

重启笔记本电脑解决了我的问题。我的客户向远程服务器发送请求，但没有及时响应，因此出现了这个错误。重新启动笔记本电脑可以刷新网络以解决这个问题。 - Vikas Piprade

Answer 1

这是Node的问题，不是Express的问题。 https://github.com/visionmedia/express/issues/1749

要解决这个问题，请将以下代码：

'Content-Length': data.length

改成：

'Content-Length': Buffer.byteLength(data)

经验法则

当你想找到字符串的内容长度时，总是使用Buffer.byteLength()

更新

我们还应该通过添加中间件来优雅地处理服务器端的错误以防止崩溃。

app.use(function (error, req, res, next) {
  if (!error) {
    next();
  } else {
    console.error(error.stack);
    res.send(500);
  }
});

- Nam Nguyen

---

你尝试过不带 Content-length 头吗？ - user568109

我尝试了不带Content-Length，也能正常工作。但是，如果我们省略Content-Length，将来会不会出现任何问题，我不确定。 - Nam Nguyen

这是个非常糟糕的想法，因为它会捕获所有未处理的异常，这可能会使你的应用程序处于未定义的状态下运行。当发生意外错误时，您应该始终关闭进程，以便您的服务有机会自我修复。 - Eric Elliott

此外，这不是Node的问题，而是一个connect问题。 - Eric Elliott

他们说：“连接或表达的问题都不是问题”，这是节点问题。我认为这是连接问题，因为很明显是连接模块引起了错误。我同意你的看法，@EricElliott。 - Nam Nguyen

Answer 2

问题在于，如果您不处理此错误并保持服务器运行状态，则可以使用此远程崩溃漏洞进行DOS攻击。但是，您可以处理它并继续执行操作，仍然在未处理异常时关闭进程（这可以防止您在未定义状态下运行 - 这是非常糟糕的事情）。

连接模块处理错误并调用next()，返回一个带有消息正文和status = 400的对象。在您的服务器代码中，您可以在express.bodyParser()之后添加此内容。

var exit = function exit() {
  setTimeout(function () {
    process.exit(1);
  }, 0);
};

app.use(function (error, req, res, next) {
  if (error.status === 400) {
    log.info(error.body);
    return res.send(400);
  }

  log.error(error);
  exit();
});

- Eric Elliott

---

我会复制你的解决方案 @Eric Elliot, ;) - Nam Nguyen

Elliot，我相信你会更新你的解决方案，因为这个错误处理函数只有在出现错误时才会被调用。所以如果（error）语句将始终为真。有什么想法吗？ - Nam Nguyen