Page MenuHomePhabricator

Aphlict - fix getActiveListenerCount return value
ClosedPublic

Authored by pmoreau on Feb 3 2015, 2:58 PM.
Tags
None
Referenced Files
F14344484: D11647.diff
Thu, Dec 19, 12:44 AM
Unknown Object (File)
Wed, Dec 18, 6:53 PM
Unknown Object (File)
Sun, Dec 8, 4:06 PM
Unknown Object (File)
Tue, Dec 3, 8:05 PM
Unknown Object (File)
Sun, Dec 1, 8:31 AM
Unknown Object (File)
Sun, Dec 1, 5:40 AM
Unknown Object (File)
Fri, Nov 29, 12:46 PM
Unknown Object (File)
Sat, Nov 23, 9:08 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
Branch
tmp
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4282
Build 4295: [Placeholder Plan] Wait for 30 Seconds

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.