Fix: Cannot read property 'statusCodes' of undefined

lib/network.js

@@ -37,7 +37,7 @@ internals.NetworkMonitor.prototype._onResponse = function (request) {
 
     var msec = Date.now() - request.info.received;
     var port = request.connection.info.port;
-    var statusCode = request.response.statusCode;
+    var statusCode = request.response && request.response.statusCode;
 
     this._responseTimes[port] = this._responseTimes[port] || { count: 0, total: 0, max: 0 };
     this._responseTimes[port].count++;
@@ -47,8 +47,10 @@ internals.NetworkMonitor.prototype._onResponse = function (request) {
         this._responseTimes[port].max = msec;
     }
 
-    this._requests[port].statusCodes[statusCode] = this._requests[port].statusCodes[statusCode] || 0;
-    this._requests[port].statusCodes[statusCode]++;
+    if (statusCode) {
+        this._requests[port].statusCodes[statusCode] = this._requests[port].statusCodes[statusCode] || 0;
+        this._requests[port].statusCodes[statusCode]++;
+    }
 };
 
 

test/network.js

@@ -69,7 +69,7 @@ describe('Network Monitor', function () {
 
                while (port) {
 
-                   expect(network._requests[port]).to.exist();
+                  expect(network._requests[port]).to.exist();
                    expect(network._requests[port].total).to.equal(20);
                    expect(network._requests[port].statusCodes[200]).to.equal(20);
 
@@ -322,4 +322,48 @@ describe('Network Monitor', function () {
             done();
         });
     });
+
+    it('does not throw if request.response is null', function (done) {
+
+        var server = new Hapi.Server();
+        server.connection({ host: 'localhost' });
+
+        server.route({
+            method: 'GET',
+            path: '/',
+            handler: function (request, reply) { reply(); }
+        });
+
+        // force response to be null to mimic client disconnect
+        server.on('response', function (request) {
+
+            request.response = null;
+        });
+
+        var network = new NetworkMonitor.Monitor(server);
+        var agent = new Http.Agent({ maxSockets: Infinity });
+        var conn = server.connections[0];
+        var usedPorts = [];
+
+        server.start(function () {
+
+            Http.get({
+                path: '/',
+                host: conn.info.host,
+                port: conn.info.port,
+                agent: agent
+            }, Hoek.ignore);
+
+            setTimeout(function () {
+
+                expect(network._requests[conn.info.port]).to.exist();
+                expect(network._requests[conn.info.port].total).to.equal(1);
+                expect(network._requests[conn.info.port].statusCodes[200]).to.not.exist();
+
+                expect(network._responseTimes[conn.info.port]).to.exist();
+
+                done();
+            }, 500);
+        });
+    });
 });