[fix] Alter logging paths to reduce memory leakage and prevent stdio issues.

Author: AvianFlu

lib/forever.js

@@ -353,6 +353,12 @@ forever.stat = function (logFile, script, callback) {
 // Starts a script with forever
 //
 forever.start = function (script, options) {
+  if (!options.uid) {
+    options.uid = options.uid || utile.randomString(4).replace(/^\-/, '_');
+  }
+  if (!options.logFile) {
+    options.logFile = forever.logFilePath(options.uid + '.log');
+  }
   return new forever.Monitor(script, options).start();
 };
 

lib/forever/monitor.js

@@ -31,7 +31,6 @@ var Monitor = exports.Monitor = function (script, options) {
   // can be attached through `monitor.use(plugin, options)`.
   //
   function bootstrap(monitor) {
-    forever.plugins.logger.attach.call(monitor, options);
 
     if (options.watch) {
       forever.plugins.watch.attach.call(monitor, options);
@@ -53,22 +52,14 @@ var Monitor = exports.Monitor = function (script, options) {
   this.childExists = false;
   this.checkFile   = options.checkFile !== false;
   this.times       = 0;
+  this.daemon      = options.daemon || false;
 
   //
   // Setup log files and logger for this instance.
   //
   this.logFile = options.logFile || path.join(forever.config.get('root'), this.uid + '.log');
   this.outFile = options.outFile;
   this.errFile = options.errFile;
-  this.logger  = options.logger || new (winston.Logger)({
-    transports: [new winston.transports.Console({ silent: this.silent })]
-  });
-
-  //
-  // Extend from the winston logger.
-  //
-  this.logger.extend(this);
-  this.log = fs.createWriteStream(this.logFile, { flags: 'a+', encoding: 'utf8', mode: '0666' });
 
   //
   // Setup restart timing. These options control how quickly forever restarts
@@ -153,6 +144,11 @@ Monitor.prototype.start = function (restart) {
     return this;
   }
 
+  if (!this.silent) {
+    child.stdout.pipe(process.stdout);
+    child.stderr.pipe(process.stderr);
+  }
+
   this.ctime = Date.now();
   this.child = child;
   this.running = true;