server onRequest handler no longer requires a socket in the request (e…

…lastic#9332) Per elastic#9302 A request sent through a HapiJS .inject() doesn't have a socket associated with the request, which causes a failure.
danwhitacre · Feb 6, 2017 · aaf5a5e · aaf5a5e
1 parent b125b44
commit aaf5a5e
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/server/http/setup_connection.js b/src/server/http/setup_connection.js
@@ -48,7 +48,9 @@ export default function (kbnServer, server, config) {
   });
 
   server.ext('onRequest', function (req, reply) {
-    if (req.raw.req.socket.encrypted) {
+    // A request sent through a HapiJS .inject() doesn't have a socket associated with the request
+    // which causes a failure.
+    if (!req.raw.req.socket || req.raw.req.socket.encrypted) {
       reply.continue();
     } else {
       reply.redirect(formatUrl({