Add option to show hostname

Author: NikxDa

README.md

@@ -56,6 +56,7 @@ This package supports several command line arguments to customize the output. A
 | `--date`, `-d` | `boolean` | `false` | Show a date in the log output. |
 | `--time`, `-t` | `boolean` | `true` | Show a time in the log output. Use `--no-time` to disable. |
 | `--pid`, `-p` | `boolean` | `false` | Show the process ID (PID) in the log output. |
+| `--hostname`, `-h` | `boolean` | `false` | Show the hostname in the log output. |
 | `--stacktrace`, `-s` | `boolean` | `true` | Show a stack trace for errors that occur. Use `--no-stacktrace` to disable. |
 | `--colorize`, `-c` | `boolean` | Depends on terminal | Colorize the console output. Use `--no-colorize` to disable. |
 | `--messageKey` | `string` | `msg` | The JSON key to read the message from. |

lib/index.js

@@ -26,6 +26,12 @@ yargs
         describe: "Show PID",
         type: "boolean"
     })
+    .option("hostname", {
+        alias: "h",
+        default: false,
+        describe: "Show hostname",
+        type: "boolean"
+    })
     .option("stacktrace", {
         alias: "s",
         default: true,

lib/test.js

@@ -15,7 +15,8 @@ const argvMock = {
     stacktrace: true,
     colorize: true,
     crlf: false,
-    pid: false
+    pid: false,
+    hostname: false
 };
 
 // Prepare a default Pino output
@@ -26,7 +27,8 @@ const pinoMock = (data = {}, messageEmpty = false) => {
         time: data.time || (new Date).getTime (),
         msg: messageEmpty ? undefined : data.message || "custom message",
         stack: data.stack || undefined,
-        pid: process.pid
+        pid: process.pid,
+        hostname: "hostname"
     });
 }
 
@@ -151,6 +153,18 @@ describe ("Command line arguments", _ => {
         done ();
     })
 
+    it ("should accept a --hostname option", done => {
+        const mock = pinoMock ();
+
+        const withHostname = parseLine (mock, { ...argvMock, hostname: true });
+        const withoutHostname = parseLine (mock, { ...argvMock, hostname: false });
+
+        assert.equal (withoutHostname.includes (`[hostname]`), false);
+        assert.equal (withHostname.includes (`[hostname]`), true);
+    
+        done ();
+    })
+
     it ("should accept a --stacktrace option", done => {
         try {
             undefinedVar1 + undefinedVar2;

lib/util.js

@@ -44,18 +44,23 @@ const formatExtras = ({
     data, 
     options: { 
         colorize: shouldColorize,
-        pid: showPid
+        pid: showPid,
+        hostname: showHostname
     } 
 }) => {
-    // Grab PID
-    const { pid } = data;
+    // Grab PID and hostname
+    const { pid, hostname } = data;
 
-    // Format the date using date-fns
-    let formattedPid = showPid ? `[${pid}]` : "";
+    // Format extras
+    const formattedPid = (showPid && pid) ? `[${pid}]` : "";
+    const formattedHostname = (showHostname && hostname) ? `[${hostname}]` : "";
 
-    // Return colored string
-    if (!shouldColorize) return formattedPid;
-    return chalk.dim (chalk.gray(formattedPid));
+    // Concatenate all extras
+    const extras = formattedPid + formattedHostname;
+
+    // Return colored strings
+    if (!shouldColorize) return extras;
+    return chalk.dim (chalk.gray(extras));
 }
 
 /**