Page MenuHomePhabricator

Allow the Aphlict server to bind to localhost
ClosedPublic

Authored by joshuaspence on Jan 8 2015, 11:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 19, 5:40 AM
Unknown Object (File)
Fri, Apr 19, 2:13 AM
Unknown Object (File)
Fri, Apr 19, 2:13 AM
Unknown Object (File)
Fri, Apr 19, 2:13 AM
Unknown Object (File)
Tue, Apr 16, 5:14 AM
Unknown Object (File)
Thu, Apr 11, 3:36 AM
Unknown Object (File)
Fri, Mar 22, 4:17 AM
Unknown Object (File)
Feb 14 2024, 11:25 PM
Subscribers

Details

Summary

If you are running the Aphlict server behind a reverse proxy (such as nginx) then there's no need to bind to 0.0.0.0. Add a --client-host flag to aphlict_server.js to allow binding to a different hostname. Also changed the other flags for consistency and clarity.

Test Plan

Started, stopped and debug the Aphlict server.

Diff Detail

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

Event Timeline

joshuaspence retitled this revision from to Allow the Aphlict server to bind to localhost.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/aphlict/management/PhabricatorAphlictManagementRestartWorkflow.php
12–16

Maybe move this spec into the superclass?

support/aphlict/server/aphlict_server.js
12–15

This cleanup is nice.

This revision is now accepted and ready to land.Jan 8 2015, 11:55 PM
src/applications/aphlict/management/PhabricatorAphlictManagementRestartWorkflow.php
12–16

Sure. This might be problematic if any of the workflows want to implement their own arguments, but we can deal with this later.

Yeah, we can always array_merge($this_args, $this->getCommonServerArgs()) or whatever.

This revision was automatically updated to reflect the committed changes.