Upgrade express-http-context (#6733)

package.json

@@ -213,7 +213,7 @@
     "dompurify": "1.0.2",
     "es6-error": "4.1.0",
     "express": "4.16.2",
-    "express-http-context": "1.1.0",
+    "express-http-context": "1.2.0",
     "extract-text-webpack-plugin": "3.0.2",
     "fastclick": "1.0.6",
     "filesize": "3.6.1",
@@ -372,8 +372,5 @@
       "maxSize": "25 kB"
     }
   ],
-  "snyk": true,
-  "browser": {
-    "express-http-context": false
-  }
+  "snyk": true
 }

src/core/logger.js

@@ -27,12 +27,6 @@ const pinoLogger = pino({
 if (config.get('enableRequestID')) {
   // eslint-disable-next-line global-require
   httpContext = require('express-http-context');
-
-  if (typeof httpContext.get !== 'function') {
-    // Set the `httpContext` to `null` so that it does not need an extra check
-    // on the browser.
-    httpContext = null;
-  }
 }
 
 export default ['debug', 'error', 'fatal', 'info', 'trace', 'warn'].reduce(

yarn.lock

@@ -5151,10 +5151,10 @@ expect@^23.1.0:
     jest-message-util "^23.1.0"
     jest-regex-util "^23.0.0"
 
-express-http-context@1.1.0:
-  version "1.1.0"
-  resolved "https://registry.yarnpkg.com/express-http-context/-/express-http-context-1.1.0.tgz#ce79a03ecab45e0fbf3b342cf5a2d39c83e34cda"
-  integrity sha512-LS47HseitRIxzBHDEQrlVwZkEkMaViM+nhRCrlWYxPNIu7W8KUZyNUOxiD93OghHesl7y+DhBYuz3yfaNHDvVA==
+express-http-context@1.2.0:
+  version "1.2.0"
+  resolved "https://registry.yarnpkg.com/express-http-context/-/express-http-context-1.2.0.tgz#8e886838f9915b57a42870b8a0a9532a82511657"
+  integrity sha512-Nw7FECgPHx0p4xE3jOP1tGYnMp+aUDQemHnuJ29TYuuAA7H3dp9zlce453CoHt7dwWNKNQG1R3vFCLnMyilnDg==
   dependencies:
     cls-hooked "^4.2.2"