【问题标题】:Node.js POST causes [Error: socket hang up] code: 'ECONNRESET'Node.js POST 导致 [错误:套接字挂起] 代码:'ECONNRESET'
【发布时间】:2013-09-12 14:36:06
【问题描述】:

我创建了一个将数据发布到休息服务的示例,我发现当我有非 ascii 或非拉丁字符(请参阅 data.firstName)时,我使用 TEST-REST.js 的发布请求将抛出

error: { [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();

在我的休息服务中,它给我带来了这样的错误:

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());,错误就会消失。

【问题讨论】:

  • 试试'Content-Type': 'application/json; charset=utf-8'
  • 你能在没有Content-length header 的情况下测试它吗?
  • 实际上这是连接问题:github.com/visionmedia/express/issues/1749。通过更改 'Content-Length' 解决: Buffer.byteLength(data)
  • 这是一个很好的收获,Nam。在您引起我注意之前,我不知道此漏洞。感谢您发布这个问题!

标签: javascript node.js express


【解决方案1】:

这是 node 的问题,不是 express 的问题。 https://github.com/visionmedia/express/issues/1749

解决,改变自

'内容长度':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);
  }
});

【讨论】:

  • 你试过没有 Content-length 标头吗?
  • 我尝试不使用 Content-length 并且效果也很好。但是,我不确定如果我们省略 Content-Length,它会在未来引起任何问题吗?
  • 这是一个非常糟糕的主意,因为它会捕获 all 未处理的异常,这可能会使您的应用程序以未定义的状态运行。当发生意外错误时,您应该始终关闭该进程,以便您的服务有机会自行修复。
  • 另外,这不是 Node 的问题。这是一个连接问题。
  • 他们说“不是连接或快递问题”,而是节点问题。我相信这是连接问题,因为很明显它在连接模块上会引发错误。我同意你@EricElliott
【解决方案2】:

问题在于,如果您不处理此错误并保持服务器处于活动状态,则此远程崩溃漏洞可用于 DOS 攻击。但是,您可以处理它并继续,当发生未处理的异常时仍然关闭进程(这会阻止您在未定义状态下运行——这是一件非常糟糕的事情)。

connect 模块处理错误并调用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 Elliot,;)
  • Elliot,我相信你会更新你的解决方案,因为这个错误处理函数只会在出现错误时被调用。所以 if (error) 语句将永远为真。有什么想法吗?
猜你喜欢
  • 2015-08-27
  • 2015-09-19
  • 1970-01-01
  • 1970-01-01
  • 2018-05-07
  • 1970-01-01
  • 2021-01-02
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多