Merge pull request #923 from iamdoron/master

remove connection event listeners when server stops
hapijs · Jun 11, 2013 · 01bec39 · 01bec39
2 parents 9a5425b + c9257dd
commit 01bec39
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/lib/server.js b/lib/server.js
@@ -246,7 +246,7 @@ internals.Server.prototype._stop = function (options, callback) {
     }, options.timeout);
 
     self.listener.close(function () {
-
+        self.listener.removeAllListeners()
         clearTimeout(timeoutId);
         callback();
     });

diff --git a/test/integration/server.js b/test/integration/server.js
@@ -154,4 +154,18 @@ describe('Server', function () {
             });
         });
     });
+    it('removes connection event listeners after it stops', function (done) {
+
+        var server = Hapi.createServer(0);
+        server.start(function () {
+            server.stop(function () {
+                server.start(function () {
+                    server.stop(function () {
+                        expect(server.listeners('connection').length).to.be.eql(0);
+                        done();
+                    });
+                });
+            });
+        });
+    });
 });