Strip colors from messages and metadata

[epd]

It would be useful to strip out the color escape codes (i.e. [33m) from the logs sent to Logstash. It looks like Winston exposes a string transform we can use here to do this: https://github.com/flatiron/winston/blob/ea8bb5b7b2c58274dc128e1942846c8f1c9c19ae/lib/winston/common.js#L129.

Would love to iterate on this if there is a better way; this is my first pass.

[jaakkos]

Thanks, I'll just push new version.

[nicobleiler]

This still seems to be a problem.

require('dotenv').config();
const winston = require('winston');
const LogstashTransport = require("winston-logstash/lib/winston-logstash-latest");
const os = require('os');

// Prepare logging
const logger = winston.createLogger({
    level: "debug",
    format: winston.format.combine(winston.format.simple(), winston.format.timestamp(), winston.format.colorize()),
    defaultMeta: {
        environment: process.env.NODE_ENV || "development",
        service: 'ephemeral',
        host: os.hostname(),
        version: require('./package.json').version,
    },
    transports: [
        new winston.transports.File({ filename: 'logs/error.log', level: 'error' }),
        new winston.transports.File({ filename: 'logs/combined.log' }),
        new winston.transports.Console(),
    ],
});

if (process.env.LOGSTASH_HOST && process.env.LOGSTASH_PORT && process.env.LOGSTASH_HOST) {
    logger.info("Using Logstash");
    logger.add(        
        new LogstashTransport({
            port: process.env.LOGSTASH_PORT,
            node_name: process.env.LOGSTASH_NODE_NAME,
            host: process.env.LOGSTASH_HOST,
            max_connect_retries: -1
        }),
    );
} else {
    logger.info("Missing ENV variables to use Logstash");
}

module.exports = {
    logger: logger
}