Page MenuHomePhabricator

Aphlict - fix getActiveListenerCount return value
ClosedPublic

Authored by pmoreau on Feb 3 2015, 2:58 PM.
Tags
None
Referenced Files
F14057828: D11647.diff
Sun, Nov 17, 7:11 AM
F14054241: D11647.id28027.diff
Fri, Nov 15, 11:15 PM
F14000310: D11647.diff
Thu, Oct 24, 9:05 PM
F14000307: D11647.id.diff
Thu, Oct 24, 9:01 PM
F14000304: D11647.id28026.diff
Thu, Oct 24, 9:00 PM
F14000300: D11647.id28027.diff
Thu, Oct 24, 8:58 PM
F13997233: D11647.diff
Thu, Oct 24, 3:06 AM
F13988293: D11647.id28026.diff
Mon, Oct 21, 2:30 PM
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.