feat: Allow logging errors when passed as message parameter

Signed-off-by: Ferdinand Thiessen <rpm@fthiessen.de>

Author: susnux

lib/ConsoleLogger.ts

@@ -5,20 +5,33 @@ export class ConsoleLogger implements ILogger {
     private context: any
 
     constructor(context: any) {
-        this.context = context
+        this.context = context || {}
     }
 
-    private formatMessage(message: string, level: LogLevel, context: any): string {
+    private formatMessage(message: string|Error, level: LogLevel, context: any): string {
         let msg = '[' + LogLevel[level].toUpperCase() + ']'
+
         if (context && context.app) {
             msg += ' ' + context.app + ': '
         }
-        return msg + message
+
+        if (typeof message === 'string') return msg + message
+
+        // basic error formatting
+        msg += `Unexpected ${message.name}`
+        if (message.message) msg += ` "${message.message}"`
+        // only add stack trace when debugging
+        if (level === LogLevel.Debug && message.stack) msg += `\n\nStack trace:\n${message.stack}`
+
+        return msg
     }
 
-    log(level: LogLevel, message: string, context: object) {
+    log(level: LogLevel, message: string|Error, context: object) {
         if (level < this.context?.level) return;
 
+        // Add error object to context
+        if (typeof message === 'object' && (context as any).error === undefined) (context as any).error = message
+
         switch (level) {
             case LogLevel.Debug:
                 console.debug(this.formatMessage(message, LogLevel.Debug, context), context)
@@ -39,23 +52,23 @@ export class ConsoleLogger implements ILogger {
         }
     }
 
-    debug(message: string, context?: object): void {
+    debug(message: string|Error, context?: object): void {
         this.log(LogLevel.Debug, message, Object.assign({}, this.context, context))
     }
 
-    info(message: string, context?: object): void {
+    info(message: string|Error, context?: object): void {
         this.log(LogLevel.Info, message, Object.assign({}, this.context, context))
     }
 
-    warn(message: string, context?: object): void {
+    warn(message: string|Error, context?: object): void {
         this.log(LogLevel.Warn, message, Object.assign({}, this.context, context))
     }
 
-    error(message: string, context?: object): void {
+    error(message: string|Error, context?: object): void {
         this.log(LogLevel.Error, message, Object.assign({}, this.context, context))
     }
 
-    fatal(message: string, context?: object): void {
+    fatal(message: string|Error, context?: object): void {
         this.log(LogLevel.Fatal, message, Object.assign({}, this.context, context))
     }
 

lib/contracts.ts

@@ -8,11 +8,11 @@ export enum LogLevel {
 
 export interface ILogger {
 
-    debug(message: string, context?: object): void
-    info(message: string, context?: object): void
-    warn(message: string, context?: object): void
-    error(message: string, context?: object): void
-    fatal(message: string, context?: object): void
+    debug(message: string|Error, context?: object): void
+    info(message: string|Error, context?: object): void
+    warn(message: string|Error, context?: object): void
+    error(message: string|Error, context?: object): void
+    fatal(message: string|Error, context?: object): void
 
 }