Page MenuHomePhabricator
Create Task
Maniphest T6903

Notifications now use WebSockets
Closed, ResolvedPublic
Actions

Assigned To
epriestley
Authored By
epriestley
Jan 8 2015, 6:16 PM
Tags
Referenced Files
None
Subscribers
aik099
btrahan
chad
• chasemp
epriestley
jayvdb
joshuaspence
richardvanvelzen
Tokens
"Mountain of Wealth" token, awarded by jayvdb."Love" token, awarded by qgil."Love" token, awarded by richardvanvelzen."Love" token, awarded by avivey."Mountain of Wealth" token, awarded by btrahan.

Description

Notifications have moved from Flash to WebSockets. These are the expected changes for administrators:

  • Upgrading
    • You will need to install the ws module for Node.js. See the documentation below for help.
    • bin/aphlict should no longer be run as root. Run as the user you want to run the server as, instead. This can be any unprivileged user.
    • bin/aphlict might not have permission to write to its PID file or log file now, if you used to run as root. Double check the settings and permissions.
  • Cleanup
    • Port 843 does not need to be accessible anymore, and can be removed from firewall rules.
    • The notifications.user option has been removed. You can delete any configuration you have.
    • The notifications.debug option has been removed. You can delete any configuration you have.
  • Supported Clients
    • Notification support has been removed for IE8 or IE9.
    • Notification support has been added for iPhones (and probably other mobile devices).

You can find details on most of these changes in the documentation, see Notifications User Guide: Setup and Configuration.

You can find technical discussion of this change in T6559.

Related Objects
Search...

Event Timeline

epriestley created this task.Jan 8 2015, 6:16 PM
epriestley raised the priority of this task from to Normal.
epriestley updated the task description. (Show Details)
epriestley added projects: Notifications, Support.
epriestley added subscribers: epriestley, joshuaspence, btrahan, chad.
• chasemp added a subscriber: • chasemp.Jan 8 2015, 7:10 PM

This is really awesome gentlemen.

chad added a comment.Jan 8 2015, 7:15 PM

Wow I just saw notifications on my iPhone

btrahan awarded a token.Jan 8 2015, 7:20 PM
avivey awarded a token.Jan 8 2015, 7:25 PM
richardvanvelzen awarded a token.Jan 8 2015, 7:56 PM
richardvanvelzen added a subscriber: richardvanvelzen.
richardvanvelzen added a comment.Jan 8 2015, 8:12 PM

The update went pretty well, and it seems to work just fine!

However, two things I noticed:

  • npm install -g ws is unlikely to work - this normally installs global scripts, but not libraries. You may want to investigate this
  • I got an error about trying to write to /var/tmp/aphlict/pid, which did not exist previously as far as I know. P1699 contains the full trace I got.

Thanks for the awesome update. Now I can spam my co-workers with fancy notifications even more. :3

qgil awarded a token.Jan 8 2015, 9:23 PM
joshuaspence added a subtask: T6559: Change notification system implementation not to require Flash.Jan 9 2015, 12:26 AM
aik099 added a subscriber: aik099.Jan 12 2015, 7:22 PM

I got an error about trying to write to /var/tmp/aphlict/pid, which did not exist previously as far as I know. P1699 contains the full trace I got.

It turned out, that Aphilist server now doesn't auto-create folder, where pid file is stored (the /var/tmp/aphlict/pid one). Try creating it and chowning to user who you're running Aphilist under.

epriestley added a comment.Jan 12 2015, 7:22 PM

See T6944: Try to create the pid directory in Aphlict.

epriestley mentioned this in 2015-01 January.Feb 2 2015, 9:56 PM
jayvdb awarded a token.Feb 5 2015, 9:15 AM
jayvdb added a subscriber: jayvdb.
epriestley closed this task as Resolved.Dec 14 2015, 6:19 PM
epriestley claimed this task.

This has been in production for quite a while.

Phabricator · Built by Phacility