Close logger

[gagle]

To close all the loggers with a callback completion (similar to the commented one):

Logger.prototype.close = function (cb){
    if (this._closed) return cb ();

    var me = this;
    var remaining = this.streams.length;
    if (!remaining) return cb ();

    this.streams.forEach (function (s){
        s.stream.stream.on ("close", function (){
            if (!--remaining && cb) cb ();
        });
        s.stream.stream.end ();
    });

    this._closed = true;
};

EDIT: this code only works when the file type is "rotation-file". If it's the normal file then s.stream.end() and s.stream.on("close") should be used.

Not tested with childs. Sorry for all the posted issues but this module needs to be revisited.

[chakrit]

I can't get this to work with a logger with 2 streams (process.stdout and rotating-file).

Is there any reliable way to close or unref all bunyan streams? Need this to properly cleanup tests and make the test runner exit :(

[chakrit]

Seems rotating-file stream is the cause, if i just leave it to process.stdout, then my test process exits cleanly.

[trentm]

Should be fixed by the above PR. This will be in bunyan 0.23.1.

[trentm]

Hrm, the unref addition in the PR only fixes this for node 0.10. Node 0.8.x doesn't have unref on setTimeout results.