From a5452f70f6d4908ecd52611d9a5e8655b755d17f Mon Sep 17 00:00:00 2001 From: Megan Walker Date: Thu, 1 Dec 2016 15:29:12 +0000 Subject: [PATCH] server onRequest handler no longer requires a socket in the request Per #9302 A request sent through a HapiJS .inject() doesn't have a socket associated with the request, which causes a failure. --- src/server/http/setup_connection.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/server/http/setup_connection.js b/src/server/http/setup_connection.js index cd922c7ca9366..736d38678972d 100644 --- 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({