Page MenuHomePhabricator

Allow Aphlict admin server to be specified
Needs ReviewPublic

Authored by joshuaspence on Jun 4 2015, 5:30 AM.
Tags
None
Referenced Files
F15523024: D13148.id31785.diff
Sun, Apr 20, 11:11 PM
F15521610: D13148.id31780.diff
Sun, Apr 20, 3:29 PM
F15518490: D13148.id.diff
Sat, Apr 19, 1:57 PM
F15516541: D13148.diff
Fri, Apr 18, 7:34 PM
F15506131: D13148.diff
Tue, Apr 15, 8:48 AM
F15486895: D13148.id.diff
Thu, Apr 10, 9:55 AM
F15418867: D13148.id31785.diff
Mar 21 2025, 1:57 AM
F15417725: D13148.id31780.diff
Mar 20 2025, 5:58 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

Allow flags to be passed to ./bin/aphlict to specify the hostname and port for the admin server. On our install, notification.server-uri is set to notify.phabricator.mydomain.com and, as such, the NodeJS server is started with --admin-host=notify.phabricator.mydomain.com instead of the desired --admin-host=0.0.0.0.

This is essentially another take on D11359.

Test Plan

Ran ./bin/aphlict debug with a bunch of different arguments.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6536
Build 6558: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

joshuaspence retitled this revision from to Allow Aphlict admin server to be specified.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.