Page MenuHomePhabricator
Differential D16254

Fix a fatal when viewing a daemon task with an invalid ID
ClosedPublic
Actions

Authored by epriestley on Jul 8 2016, 11:33 AM.
Tags
None
Referenced Files
F18783759: D16254.id.diff
Mon, Oct 13, 9:20 AM
F18783598: D16254.id39102.diff
Mon, Oct 13, 8:21 AM
F18783316: D16254.id39102.diff
Mon, Oct 13, 5:57 AM
F18750276: D16254.id39096.diff
Sat, Oct 4, 7:12 AM
F18685180: D16254.diff
Fri, Sep 26, 10:42 AM
F18629302: D16254.id39096.diff
Tue, Sep 16, 8:27 AM
F18616421: D16254.diff
Sun, Sep 14, 5:28 PM
F18599456: D16254.id.diff
Sep 13 2025, 7:28 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T11295: Worker queue 404 page fails with invalid `$header`
Commits
rP1c088822b4d2: Fix a fatal when viewing a daemon task with an invalid ID
Summary

Fixes T11295. Prior to this change, the "404 page" for daemon tasks fatals.

This page is special cased a little bit and not a normal 404 page, because it's possible for you to click a valid link and the task to get GC'd by the time you load the page, or similar. It tries to be a little more user-friendly than a bare 404.

Test Plan
  • Visited /daemon/task/1428348920328/ (any invalid ID).
  • Now got a nice "no such task" page.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 39096.Jul 8 2016, 11:33 AM
epriestley retitled this revision from to Fix a fatal when viewing a daemon task with an invalid ID.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T11295: Worker queue 404 page fails with invalid `$header`.
chad accepted this revision.Jul 8 2016, 2:08 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Jul 8 2016, 2:08 PM
Closed by commit rP1c088822b4d2: Fix a fatal when viewing a daemon task with an invalid ID (authored by epriestley, committed by epriestley). · Explain WhyJul 8 2016, 2:08 PM
This revision was automatically updated to reflect the committed changes.
• fdxchina added a parent revision: D16312: Allow file comments to be edited.Jul 21 2016, 2:44 AM

Revision Contents
Changeset List

PathSize
src/
applications/
daemon/
controller/
14 lines

Diff 39102

View Options

src/applications/daemon/controller/PhabricatorWorkerTaskDetailController.php

Loading...
Phabricator · Built by Phacility