Tighten up Phragment routes to prevent page crashes
ClosedPublic
Actions

Authored by hach-que on Jun 20 2014, 12:45 AM.

Tags

None

Referenced Files

	F15528019: D9635.id23118.diff
	Tue, Apr 22, 8:23 AM

	F15517091: D9635.diff
	Sat, Apr 19, 12:00 AM

	F15488526: D9635.id23118.diff
	Thu, Apr 10, 11:20 PM

	F15486792: D9635.id23115.diff
	Thu, Apr 10, 9:42 AM

	F15480505: D9635.id23118.diff
	Tue, Apr 8, 2:31 PM

	F15478702: D9635.id23115.diff
	Tue, Apr 8, 2:16 AM

	F15477709: D9635.id23115.diff
	Mon, Apr 7, 5:25 PM

	F15477370: D9635.id.diff
	Mon, Apr 7, 1:56 PM

Subscribers

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T5418: Phragment policy controller fatals when given no arguments

Commits

Restricted Diffusion Commit
rP5013969a54b0: Tighten up Phragment routes to prevent page crashes

Summary

Fixes T5418. These routes were a little more permissive than they should have been.

Test Plan

Hit those URLs without a path, got a 404 instead.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

hach-que updated this revision to Diff 23115.Jun 20 2014, 12:45 AM

hach-que retitled this revision from to Tighten up Phragment routes to prevent page crashes.

hach-que updated this object.

hach-que edited the test plan for this revision. (Show Details)

hach-que added a reviewer: epriestley.

hach-que added a task: T5418: Phragment policy controller fatals when given no arguments.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 20 2014, 12:45 AM

Herald added subscribers: hach-que, Korvin, epriestley. · View Herald Transcript

Harbormaster completed remote builds in B1254: Diff 23115.Jun 20 2014, 12:46 AM

epriestley accepted this revision.Jun 20 2014, 1:42 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 20 2014, 1:42 AM

Closed by commit rP5013969a54b0 (authored by @hach-que).

Revision Contents
Changeset List

Path

Size

src/

applications/

phragment/

application/

PhabricatorApplicationPhragment.php

10 lines

Diff 23118

View Options

src/applications/phragment/application/PhabricatorApplicationPhragment.php

Tighten up Phragment routes to prevent page crashesClosedPublicActions