Merge pull request #1912 from gradido/use-logger-categories

refactor: Use Logger Categories

backend/log4js-config.json

@@ -13,6 +13,14 @@
     {
       "type": "dateFile",
       "filename": "../logs/backend/apollo.log",
+      "pattern": "%d{ISO8601} %p %c %m",
+      "keepFileExt" : true,
+      "fileNameSep" : "_" 
+    },
+    "backend": 
+    {
+      "type": "dateFile",
+      "filename": "../logs/backend/backend.log",
       "pattern": "%d{ISO8601} %p %c %X{user} %f:%l %m",
       "keepFileExt" : true,
       "fileNameSep" : "_" 
@@ -31,24 +39,52 @@
       "level": "error",
       "appender": "errorFile"
     },
-    "out": 
+    "out":
     {
       "type": "stdout",
       "layout": 
       {
         "type": "pattern", "pattern": "%d{ISO8601} %p %c %X{user} %f:%l %m"
       }
-
-    }
+    },
+    "apolloOut": 
+    {
+      "type": "stdout",
+      "layout": 
+      {
+        "type": "pattern", "pattern": "%d{ISO8601} %p %c %m"
+      }
+    }    
   },
-  "categories": 
+  "categories":
   {
     "default": 
     {
       "appenders": 
       [
         "out",
+        "errors"
+      ],
+      "level": "debug",
+      "enableCallStack": true
+    },
+    "apollo":
+    {
+      "appenders":
+      [
         "apollo",
+        "apolloOut",
+        "errors"
+      ],
+      "level": "debug",
+      "enableCallStack": true
+    },
+    "backend":
+    {
+      "appenders":
+      [
+        "backend",
+        "out",
         "errors"
       ],
       "level": "debug",

backend/src/server/logger.ts

@@ -12,7 +12,6 @@ log4js.configure(options)
 const apolloLogger = log4js.getLogger('apollo')
 const backendLogger = log4js.getLogger('backend')
 
-apolloLogger.addContext('user', 'unknown')
 backendLogger.addContext('user', 'unknown')
 
 export { apolloLogger, backendLogger }

backend/src/server/plugins.ts

@@ -32,16 +32,16 @@ const logPlugin = {
   requestDidStart(requestContext: any) {
     const { logger } = requestContext
     const { query, mutation, variables } = requestContext.request
-    logger.trace(`Request:
+    logger.info(`Request:
 ${mutation || query}variables: ${JSON.stringify(filterVariables(variables), null, 2)}`)
     return {
       willSendResponse(requestContext: any) {
-        if (requestContext.context.user) logger.trace(`User ID: ${requestContext.context.user.id}`)
+        if (requestContext.context.user) logger.info(`User ID: ${requestContext.context.user.id}`)
         if (requestContext.response.data)
-          logger.trace(`Response-Data:
+          logger.info(`Response-Data:
 ${JSON.stringify(requestContext.response.data, null, 2)}`)
         if (requestContext.response.errors)
-          logger.trace(`Response-Errors:
+          logger.error(`Response-Errors:
 ${JSON.stringify(requestContext.response.errors, null, 2)}`)
         return requestContext
       },