screwdriver-api 8.0.14 → 8.0.15

package/lib/server.js

@@ -9,12 +9,11 @@ process.on('unhandledRejection', (reason, p) => {
 });
 
 /**
- * If we're throwing errors, let's have them say a little more than just 500
- * @method prettyPrintErrors
+ * @method handlePreResponseLogs
  * @param  {Hapi.Request}    request Hapi Request object
  * @param  {Hapi.h}     h   Hapi Response Toolkit
  */
-function prettyPrintErrors(request, h) {
+function handlePreResponseLogs(request, h) {
     const { response } = request;
     const { release } = request.server.app;
 
@@ -22,31 +21,45 @@ function prettyPrintErrors(request, h) {
         h.state(release.cookieName, release.cookieValue);
     }
 
-    if (!response.isBoom) {
-        return h.continue;
-    }
+    // Pretty print errors
+    if (response.isBoom) {
+        const err = response;
+        const errName = err.output.payload.error;
+        const errMessage = err.message;
+        const { statusCode } = err.output.payload;
+        const stack = err.stack || errMessage;
+
+        // If we're throwing errors, let's have them say a little more than just 500
+        if (statusCode === 500) {
+            request.log(['server', 'error'], stack);
+        }
 
-    const err = response;
-    const errName = err.output.payload.error;
-    const errMessage = err.message;
-    const { statusCode } = err.output.payload;
-    const stack = err.stack || errMessage;
+        const res = {
+            statusCode,
+            error: errName,
+            message: errMessage
+        };
 
-    if (statusCode === 500) {
-        request.log(['server', 'error'], stack);
-    }
+        if (err.data) {
+            res.data = err.data;
+        }
 
-    const res = {
-        statusCode,
-        error: errName,
-        message: errMessage
-    };
+        return h.response(res).code(statusCode);
+    }
 
-    if (err.data) {
-        res.data = err.data;
+    // Log request payload when it takes longer than 5 seconds to respond
+    // This is to prevent logging payloads for every request
+    if (request.info && request.info.received && Date.now() - request.info.received > 5000 && request.payload) {
+        request.log(['payload'], {
+            method: request.method,
+            path: request.path,
+            payload: request.payload,
+            statusCode: request.response && request.response.statusCode,
+            responseTime: Date.now() - request.info.received
+        });
     }
 
-    return h.response(res).code(statusCode);
+    return h.continue;
 }
 
 /**
@@ -164,8 +177,7 @@ module.exports = async config => {
             });
         }
 
-        // Write prettier errors
-        server.ext('onPreResponse', prettyPrintErrors);
+        server.ext('onPreResponse', handlePreResponseLogs);
 
         // Audit log
         if (config.log && config.log.audit.enabled) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "screwdriver-api",
-  "version": "8.0.14",
+  "version": "8.0.15",
   "description": "API server for the Screwdriver.cd service",
   "main": "index.js",
   "scripts": {