Homec4science

Improve Aphlict server

Authored by epriestley <git@epriestley.com> on Jun 14 2012, 15:12.

Description

Improve Aphlict server

Summary:

  • Move to port 22280 by default.
  • Warn when running as non-root.
  • Allow subscription and publish/admin ports to be configured.
  • Allow server to drop root after binding to 843.
  • Allow log path to be configured.
  • Add /status/ admin URI which shows server status.
  • Return HTTP 400 Bad Request for other requests, instead of hanging.
  • Minor formatting cleanup.

Test Plan:
Ran without root:

$ node aphlict_server.js

...got a good error message. Ran with --user:

$ sudo node aphlict_server.js --user=epriestley

...verified server dropped permissions. Ran with --port / --admin. Hit /status/ with GET, got status. Hit other URLs with GET, got 400.

Reviewers: allenjohnashton, ddfisher, keebuhm

Reviewed By: ddfisher

CC: aran

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

Details

Committed
epriestley <git@epriestley.com>Jun 14 2012, 15:12
Pushed
aubortJan 31 2017, 17:16
Parents
rPH9a4243b4b3df: Minor, fix MailImplementationTestAdapter constructor. See D2706.
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH86040227b0b5: Improve Aphlict server (authored by epriestley <git@epriestley.com>).Jun 14 2012, 15:12