[server/logging] fix the order we expect tags to be in

spalger · spalger · commit 76e09e67400a · 2016-10-20T14:04:58.000-07:00
diff --git a/src/server/logging/__tests__/log_interceptor.js b/src/server/logging/__tests__/log_interceptor.js
@@ -10,7 +10,7 @@ function stubEconnresetEvent() {
     event: 'error',
     pid: 1234,
     timestamp: Date.now(),
-    tags: ['error', 'client', 'connection'],
+    tags: ['connection', 'client', 'error'],
     data: error
   };
 }
diff --git a/src/server/logging/log_interceptor.js b/src/server/logging/log_interceptor.js
@@ -24,7 +24,7 @@ export class LogInterceptor extends Stream.Transform {
    *  @param {object} - log event
    */
   downgradeIfEconnreset(event) {
-    const isClientError = doTagsMatch(event, ['error', 'client', 'connection']);
+    const isClientError = doTagsMatch(event, ['connection', 'client', 'error']);
     const isEconnreset = isClientError && get(event, 'data.errno') === 'ECONNRESET';
 
     if (!isEconnreset) return null;

Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@ function stubEconnresetEvent() {`
`10`	`10`	`event: 'error',`
`11`	`11`	`pid: 1234,`
`12`	`12`	`timestamp: Date.now(),`
`13`		`- tags: ['error', 'client', 'connection'],`
	`13`	`+ tags: ['connection', 'client', 'error'],`
`14`	`14`	`data: error`
`15`	`15`	`};`
`16`	`16`	`}`