Move message posting to PhabricatorNotificationClient

Summary: Ref T4324. Centralize communication with the notification server. This will probably get less messy eventually.

Test Plan: Posted some messages.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T4324

Differential Revision: https://secure.phabricator.com/D8252

src/applications/feed/PhabricatorFeedStoryPublisher.php

@@ -161,16 +161,16 @@ final class PhabricatorFeedStoryPublisher {
   }
 
   private function sendNotification($chrono_key) {
-    $server_uri = PhabricatorEnv::getEnvConfig('notification.server-uri');
 
     $data = array(
       'key' => (string)$chrono_key,
     );
 
-    id(new HTTPSFuture($server_uri, $data))
+    try {
-      ->setMethod('POST')
+      PhabricatorNotificationClient::postMessage($data);
-      ->setTimeout(1)
+    } catch (Exception $ex) {
-      ->resolve();
+      // Ignore, these are not critical.
+    }
   }
 
   /**

src/applications/notification/client/PhabricatorNotificationClient.php

@@ -25,4 +25,13 @@ final class PhabricatorNotificationClient {
     return $status;
   }
 
+  public static function postMessage(array $data) {
+    $server_uri = PhabricatorEnv::getEnvConfig('notification.server-uri');
+
+    id(new HTTPSFuture($server_uri, $data))
+      ->setMethod('POST')
+      ->setTimeout(1)
+      ->resolvex();
+  }
+
 }