From 4626fb4ef00f8da35493d480d8535032f76d9e76 Mon Sep 17 00:00:00 2001 From: Joshua Spence Date: Sat, 31 Oct 2015 11:20:23 +0000 Subject: [PATCH] Update "should not run as root" message Summary: These should be fine to land whenever. Test Plan: N/A Reviewers: #blessed_reviewers, epriestley Reviewed By: #blessed_reviewers, epriestley Subscribers: Korvin Differential Revision: https://secure.phabricator.com/D14066 --- .../management/PhabricatorAphlictManagementWorkflow.php | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php b/src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php index 7747a73866..659fb80edb 100644 --- a/src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php +++ b/src/applications/aphlict/management/PhabricatorAphlictManagementWorkflow.php @@ -132,10 +132,7 @@ abstract class PhabricatorAphlictManagementWorkflow if (posix_getuid() == 0) { throw new PhutilArgumentUsageException( - pht( - // TODO: Update this message after a while. - 'The notification server should not be run as root. It no '. - 'longer requires access to privileged ports.')); + pht('The notification server should not be run as root.')); } // Make sure we can write to the PID file.