log4js for Node doesn't output to file, only to console

Question

I need a basic logging system for a NodeJS application, so I'm doing some tests with log4js, which seems to be the standard for these cases. I need to print the messages both to the console and a file, so I wrote this code:

// Logger
var log4js = require('log4js');

log4js.configure({
  appenders: {
    'console': { type: 'console' },
    'file': { type: 'file', filename: 'logs/mailer.log' }
  },
  categories: {
    default: { appenders: ['file', 'console'], level: 'DEBUG' },
  }
});

var logger = log4js.getLogger("Mailer");

logger.info("Starting application");

try {
  // CODE TO READ APP CONFIG FILE
}
catch(e) {
  logger.error("Couldn't read app configuration file (config.yml)");

  // This is the trouble maker. It kills the app without flushing the logs
  process.exit(1);
}

When I run the application, the logs appear like this:

[2019-07-29T16:07:24.763] [INFO] Mailer - Starting application

The problem is that I can see the messages in the console, but the log file remains empty. If I delete it and run the application again, it's created, so I suppose that the problem is a missing option or something like that.

Any suggestion?

I used your code verbatim, and it writes to the file perfectly fine. Make sure it isn't opened elsewhere or the like. — user5734311
– user5734311, Commented Jul 29, 2019 at 14:26
I found the reason of the problem: after I print the error message, I quit the application by running process.exit(1);, and it seems that the logger doesn't flush the messages to the file. I'm doing a little research and I'll post the solution when I find it. — Fel
– Fel, Commented Jul 29, 2019 at 14:57
Yeah, I guess the logger writes asynchronously, so when you kill the process right after logging something, it probably doesn't get saved. You should add this to the code in your question though so we can reproduce this. — user5734311
– user5734311, Commented Jul 29, 2019 at 15:00

Fel · Accepted Answer · 2019-07-30 08:37:18Z

6

To avoid changing the application logic and all the process.exit() calls, I've replaced the file appender by its synchronous version fileSync and now everything works as expected. So, the only thing I changed is the line where I declare the 'file' appender:

    'file': { type: 'fileSync', filename: 'logs/mailer.log' }

Cheers!

answered Jul 30, 2019 at 8:37

Fel

4,89812 gold badges46 silver badges102 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Bigyan Devkota Over a year ago

works like a charm, i still get confused with async programming

alesssz · Accepted Answer · 2019-07-29 14:22:05Z

-1

You could use loggerWinston...I used it in my nodejs application and it works fine.
You should add it in your server.js (the file you use to start node)

const winston = require('winston');

const tsFormat = () => (new Date()).toLocaleTimeString();
global.loggerWinston = new (winston.Logger)({
  transports: [
    // colorize the output to the console
    new (winston.transports.Console)({
      timestamp: tsFormat,
      colorize: true,
      level: 'debug'
    }),
    new (winston.transports.File)({
      filename: 'results.log',
      timestamp: tsFormat,
//      level: (process.env.NODE_ENV || 'development') === 'development' ? 'debug' : 'info'
      level: 'info'
    })
  ]
});

The results.log file should be added in your project folder.

This is the reference https://www.npmjs.com/package/winston

answered Jul 29, 2019 at 14:22

alesssz

5181 gold badge7 silver badges26 bronze badges

2 Comments

Fel Over a year ago

Thanks for the suggestion, I didn't know this library, I'll have a look to see its features.

alesssz Over a year ago

@Fel sure thing, let me know if it works the way you want it :)

Collectives™ on Stack Overflow

log4js for Node doesn't output to file, only to console

2 Answers 2

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related