` for Aphlict
ClosedPublic
Actions

Authored by joshuaspence on Jan 2 2015, 1:13 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

Restricted Diffusion Commit
rP9f31e023f425: Minor improvements for handling of `/status/` for Aphlict

Summary

We don't need to handle any request data for the /status/ route, so we can simplify this code slightly.

Test Plan

> curl http://127.0.0.1:22281/status/
{"uptime":2543,"clients.active":0,"clients.total":0,"messages.in":0,"messages.out":0,"log":"/var/log/aphlict.log","version":6}

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 26757.Jan 2 2015, 1:13 PM

joshuaspence retitled this revision from to Minor improvements for handling of `/status/` for Aphlict.

joshuaspence updated this object.

joshuaspence edited the test plan for this revision. (Show Details)

joshuaspence added a reviewer: epriestley.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 2 2015, 1:13 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley accepted this revision.Jan 2 2015, 4:45 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jan 2 2015, 4:45 PM

Closed by commit rP9f31e023f425: Minor improvements for handling of `/status/` for Aphlict (authored by joshuaspence, committed by joshuaspence). · Explain WhyJan 2 2015, 10:11 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

support/

aphlict/

server/

aphlict_server.js

33 lines

Diff 26796

View Options

support/aphlict/server/aphlict_server.js

Minor improvements for handling of `/status/` for AphlictClosedPublicActions