FileLoggerAdapter.getLogs does not get the latest log messages

[blacha]

When logging out a number of messages, when doing a query ordering by desc, will return the oldest messages then order them by time.

var fileLogger = new FileLoggerAdapter();
for (var i = 0; i < 10; i ++) {
    fileLogger.info(i);
}

// Wait for the logs to flush before querying 
setTimeout(function() {
    fileLogger.query({size:2, order: 'desc'})
        .then((res)=> {
            console.log(res)
        });
}, 100);

In this output res will be

[ { level: 'info',
    message: '1',
    timestamp: '2016-04-20T07:12:53.376Z' },
  { level: 'info',
    message: '0',
    timestamp: '2016-04-20T07:12:53.375Z' } ]

Its not until the size is expanded to all of the log messages that it gets them all in the right order.

Ideally this should return

[ { level: 'info',
    message: '9',
    timestamp: '2016-04-20T07:12:53.377Z' },
  { level: 'info',
    message: '8',
    timestamp: '2016-04-20T07:12:53.377Z' } ]

[blacha]

Looks like this is a problem with rotating file logger

https://github.com/winstonjs/winston-daily-rotate-file/blob/master/index.js#L305

Will break out of the query when the number of rows reach the limit, in my case two rows. Then it will reverse it in the close

https://github.com/winstonjs/winston-daily-rotate-file/blob/master/index.js#L288

[hramos]

We're closing this issue due to inactivity.

If this is a bug you care about that is not getting attention, consider opening a pull request with a fix.