Skip to content

Commit 9e0f70e

Browse files
author
epriestley
committedJan 8, 2015
Rewrite Aphlict to use Websockets
Summary: Fixes T6559. No more flash, use Websockets. This is less aggressive than the earlier version, and retains more server logic. - Support "wss". - Make the client work. - Remove "notification.user" entirely. - Seems ok? Test Plan: In Safari, Firefox and Chrome, saw the browsers connect. Made a bunch of comments/updates and saw notifications. Notable holes in the test plan: - Haven't tested "wss" yet. I'll do this on secure. - Notifications are //too fast// now, locally. I get them after I hit submit but before the page reloads. - There are probably some other rough edges, this is a fairly big patch. Reviewers: joshuaspence, btrahan Reviewed By: joshuaspence, btrahan Subscribers: fabe, btrahan, epriestley Maniphest Tasks: T6713, T6559 Differential Revision: https://secure.phabricator.com/D11143
1 parent 9a8eb40 commit 9e0f70e

31 files changed

+395
-2777
lines changed
 

‎.gitignore

+3
Original file line numberDiff line numberDiff line change
@@ -30,3 +30,6 @@
3030

3131
# User extensions
3232
/src/extensions/*
33+
34+
# NPM local packages
35+
/support/aphlict/server/node_modules/

‎externals/vegas/LICENSE

-581
This file was deleted.

‎externals/vegas/README

-29
This file was deleted.

‎externals/vegas/src/system/Serializer.as

-86
This file was deleted.

‎externals/vegas/src/vegas/strings/JSON.as

-107
This file was deleted.

‎externals/vegas/src/vegas/strings/json/JSONError.as

-84
This file was deleted.

0 commit comments

Comments
 (0)
Failed to load comments.