1

我正在使用 winston 制作一个自我记录器，它允许我在记录中添加时间戳和行号。代码可以完成每个功能，但是当组合时，它们不能按预期工作。

// **help to add timestamp**
var logger = new (winston.Logger)({
  transports : [new (winston.transports.Console)({
    json : false,
    timestamp : true,
    colorize: true
  }), new winston.transports.File({
    filename : __dirname + '/debug.log',
    json : true
  })]
  ,exitOnError : false
});

// **help me to add line number**
var logger_info_old = winston.info;
logger.info = function(msg) {
    var fileAndLine = traceCaller(1);
    return logger_info_old.call(this, fileAndLine + ":" + msg);
}

但是，当添加行号配置时，日志记录的时间戳将消失。

比如添加行号之前的配置。

logger.info("abc");
2013-11-24T09:49:15.914Z - info:339:abc

添加行号配置时

logger.info("abc");
info: (H:\Dropbox\node\fablab\utils\logging.js:85:abc

我想要的最佳结果就像

logger.info("abc");
2013-11-24T09:49:15.914Z - info: (H:\Dropbox\node\fablab\app.js:339:abc

我可以解决这个问题吗？

4

2 回答 2

3

我得到了这个工作，这就是我做的。

var transports = [
  new(winston.transports.Console)({
    colorize: true,
    prettyPrint: true,
    timestamp : true,
    level: 'debug',
  })
];

var log = new(winston.Logger)({
  "transports": transports
});

for (var func in winston.levels) {
  var oldFunc = log[func];

  log[func] = function() {
    var args = Array.prototype.slice.call(arguments);
    args.unshift(traceCaller(1));
    oldFunc.apply(log, args);
  }
}

有了这个，我得到了时间戳和文件。（注意 traceCaller(1) 来自这个 stackoverflow 问题：我想在日志语句中显示文件名）。我在 winston.levels 上执行了 for 循环，所以我会拾取所有功能，而不仅仅是信息。

你的不工作的原因是你的 logger_info_old 来自winston而不是来自logger。所以

var logger_info_old = winston.info;

本来应该

var logger_info_old = logger.info;
于 2013-12-06T19:18:01.597 回答
3

我更新了@jeff-whiting 的答案（使用闭包并修复字符串插值）并使其成为一个函数。
传递现有记录器以将呼叫站点信息添加到其级别函数。
注意logger.log()没有改变，只是logger.{level}()改变了。

// add callsite info to winston logger instance
function addCallSite(logger) {
  // WARNING: traceCaller is slow
  // http://stackoverflow.com/a/20431861/665507
  // http://stackoverflow.com/a/13411499/665507
  /**
  * examines the call stack and returns a string indicating
  * the file and line number of the n'th previous ancestor call.
  * this works in chrome, and should work in nodejs as well.
  *
  * @param n : int (default: n=1) - the number of calls to trace up the
  *   stack from the current call.  `n=0` gives you your current file/line.
  *  `n=1` gives the file/line that called you.
  */
  function traceCaller(n) {
    if( isNaN(n) || n<0) n=1;
    n+=1;
    var s = (new Error()).stack
      , a=s.indexOf('\n',5);
    while(n--) {
      a=s.indexOf('\n',a+1);
      if( a<0 ) { a=s.lastIndexOf('\n',s.length); break;}
    }
    b=s.indexOf('\n',a+1); if( b<0 ) b=s.length;
    a=Math.max(s.lastIndexOf(' ',b), s.lastIndexOf('/',b));
    b=s.lastIndexOf(':',b);
    s=s.substring(a+1,b);
    return s;
  }

  // assign to `logger.{level}()`
  for (var func in logger.levels) {
    (function (oldFunc) {
      logger[func] = function() {
        var args = Array.prototype.slice.call(arguments);
        if (typeof args[0] === 'string') {
          args[0] = traceCaller(1) + ' ' + args[0];
        }
        else {
          args.unshift(traceCaller(1));
        }
        oldFunc.apply(logger, args);
      };
    })(logger[func]);
  };
}
于 2014-09-12T18:13:29.080 回答