0

我是 nodejs 的新手，我正在尝试创建一个用于自定义格式化日志记录到文件和控制台的函数。以下是我为此目的使用的功能。

module.exports.log =function(req,res) 
{

res.on('finish', function() 
{

point = new Date(); 
clientIP = req.ip;
request = req.protocol + '://' + req.get('host') + req.originalUrl;
userAgent = req.get('User-Agent');
method = req.method;
statusCode = res.statusCode;


output =
"["
+point.toDateString()
+"] -- ["
+point.getHours()
+":"
+point.getMinutes()
+":"
+point.getSeconds()
+"] -- [client IP : "
+clientIP
+"] -- [user agent : "
+userAgent
+"] -- [method : "
+method
+"] -- [request : "
+request
+"] -- [response code : "
+statusCode
+"]"
;

console.log(output);
var fs = require('fs');
fs.appendFile('QAB.log', output, function (err) 
{
    if (err) throw err;
});

})

}

不知道这是否是正确的做法。我检查了winston、log4js-node等库。但我只是想我会自己创建它。我想知道如果我使用winston、log4js或其他东西是否会获得性能优势。

或者换一种说法……我做错了吗？这会影响我的节点服务器的性能吗？

4

1 回答 1

1

编写自己的记录器没有任何问题，因为编写任何模拟任何 npm 模块的定制代码都没有问题。

出于同样的原因，我可能会建议您不要编写自己的身份验证或安全中间件。这些问题已经解决了，并且足够通用，每个人都可以使用。

在每个日志语句中写入文件非常昂贵，因为写入磁盘会增加请求的处理时间。

此外，如果您正在写入文件，如果您正在考虑将所有日志存储在那里，则需要在某个时候轮换文件，如果可以的话，最好转到 syslog。

像最流行的winston（https://www.npmjs.com/package/winston）这样的记录器可以添加一个传输，它将执行您所追求的自定义日志记录，并以最快的方式写入文件。

如果您追求速度，那么 pico 非常好（https://github.com/pinojs/pino）并且速度更快，具体取决于您将日志发送到的位置。

于 2018-07-06T16:39:24.383 回答