Page MenuHomePhabricator

Aphlict - fix getActiveListenerCount return value
ClosedPublic

Authored by pmoreau on Feb 3 2015, 2:58 PM.
Tags
None
Referenced Files
F19932760: D11647.diff
Fri, Apr 10, 8:13 PM
F19926554: D11647.id28027.diff
Tue, Apr 7, 5:02 PM
F19925690: D11647.id28026.diff
Tue, Apr 7, 12:25 PM
F19925688: D11647.id28026.diff
Tue, Apr 7, 12:24 PM
F19923246: D11647.id.diff
Tue, Apr 7, 1:38 AM
F19917653: D11647.id28026.diff
Sat, Apr 4, 8:03 AM
F19878686: D11647.diff
Tue, Mar 17, 4:21 AM
F19868851: D11647.diff
Sun, Mar 15, 4:10 AM
Subscribers

Details

Summary

Ref T7126. Dictionaries do not have a length property unlike arrays resulting in the getActiveListenerCount() function returning undefined results. Using the length property on the array of keys will work.

Test Plan

Using wscat to generate multiple connections to the server, establish new ones and close others while keeping an eye on the displayed clients.active value.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

pmoreau retitled this revision from to Aphlict - fix getActiveListenerCount return value.
pmoreau updated this object.
pmoreau edited the test plan for this revision. (Show Details)
pmoreau added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Feb 3 2015, 3:00 PM
This revision was automatically updated to reflect the committed changes.