16

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

错误:{ [错误:套接字挂起]代码:'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());,错误就会消失。

4

2 回答 2

43

这是节点的问题,而不是快递的问题。https://github.com/visionmedia/express/issues/1749

解决,改变

“内容长度”:data.length

“内容长度”:Buffer.byteLength(数据)

经验法则

当您想查找字符串的内容长度时,请始终使用Buffer.byteLength()

更新

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

app.use(function (error, req, res, next) {
  if (!error) {
    next();
  } else {
    console.error(error.stack);
    res.send(500);
  }
});
于 2013-09-09T07:26:41.343 回答
1

问题是,如果您不处理此错误并保持服务器处于活动状态,则此远程崩溃漏洞可用于 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();
});
于 2013-09-12T22:35:28.717 回答