Page MenuHomePhabricator

Implement unsubscription in the Aphlict client.
ClosedPublic

Authored by joshuaspence on Jun 11 2014, 11:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, May 3, 4:09 AM
Unknown Object (File)
Mon, Apr 29, 3:23 PM
Unknown Object (File)
Wed, Apr 24, 10:24 PM
Unknown Object (File)
Sat, Apr 20, 7:04 PM
Unknown Object (File)
Fri, Apr 19, 4:35 AM
Unknown Object (File)
Mar 5 2024, 12:33 PM
Unknown Object (File)
Feb 25 2024, 9:23 PM
Unknown Object (File)
Feb 14 2024, 9:16 PM
Subscribers

Details

Summary

Ref T5284. When an AphlictClient is closed, it will eventually be purged from the pool by the AphlictMaster. When this happens, also unsubscribe the purged client from all notifications, and send an unsubscribe command to the Aphlict server if possible.

Test Plan

Verified the output of the Aphlict server (running in debug mode):

[Wed Jun 11 2014 23:21:31 GMT+0000 (UTC)] <FlashPolicy> Policy Request From ::ffff:192.168.1.1
[Wed Jun 11 2014 23:21:31 GMT+0000 (UTC)] <Listener/2> Connected from ::ffff:192.168.1.1
[Wed Jun 11 2014 23:21:31 GMT+0000 (UTC)] <Listener/2> Received data: {"command":"subscribe","data":["PHID-USER-cb5af6p4oepy5tlgqypi"]}
[Wed Jun 11 2014 23:21:31 GMT+0000 (UTC)] <Listener/2> Subscribed to: ["PHID-USER-cb5af6p4oepy5tlgqypi"]
[Wed Jun 11 2014 23:21:39 GMT+0000 (UTC)] <Listener/2> Received data: {"command":"subscribe","data":["PHID-TASK-l2dtbs5xrt2b7abgh5a6"]}
[Wed Jun 11 2014 23:21:39 GMT+0000 (UTC)] <Listener/2> Subscribed to: ["PHID-TASK-l2dtbs5xrt2b7abgh5a6"]
[Wed Jun 11 2014 23:21:57 GMT+0000 (UTC)] <Listener/2> Received data: {"command":"unsubscribe","data":["PHID-TASK-l2dtbs5xrt2b7abgh5a6"]}
[Wed Jun 11 2014 23:21:57 GMT+0000 (UTC)] <Listener/2> Unsubscribed from: ["PHID-TASK-l2dtbs5xrt2b7abgh5a6"]

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Implement unsubscription in the Aphlict client..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 11 2014, 11:35 PM
epriestley updated this revision to Diff 22749.

Closed by commit rPcbd95b91b5f1 (authored by @joshuaspence, committed by @epriestley).