Differential D19155

Remove defunct "download" route in Files pointing to nonexistent controller
ClosedPublic
Actions

Authored by epriestley on Feb 28 2018, 11:21 PM.

Tags

None

Referenced Files

	F15474791: D19155.diff
	Sun, Apr 6, 12:40 PM

	F15458258: D19155.id45886.diff
	Sun, Mar 30, 10:15 PM

	F15458257: D19155.id45890.diff
	Sun, Mar 30, 10:15 PM

	F15457898: D19155.diff
	Sun, Mar 30, 7:57 PM

	F15455324: D19155.id45890.diff
	Sat, Mar 29, 11:31 PM

	F15449635: D19155.id.diff
	Fri, Mar 28, 11:22 AM

	F15433369: D19155.id45886.diff
	Mon, Mar 24, 10:51 PM

	F15431522: D19155.id45890.diff
	Mon, Mar 24, 12:51 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13094: Improve file behaviors around POST requests and downloads
Commits: rPafc98f5d5d08: Remove defunct "download" route in Files pointing to nonexistent controller

Summary

Depends on D19154. Ref T13094. This controller was removed at some point and this route no longer works.

I plan to add a new download/ route to let us tighten the form-action Content Security Policy.

Test Plan

Grepped for the route and controller, no hits.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 28 2018, 11:21 PM

Harbormaster completed remote builds in B19697: Diff 45886.Feb 28 2018, 11:23 PM

epriestley requested review of this revision.Feb 28 2018, 11:23 PM

epriestley added a child revision: D19156: Never generate file download forms which point to the CDN domain, tighten "form-action" CSP.Mar 1 2018, 12:55 AM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 1 2018, 1:19 AM

Closed by commit rPafc98f5d5d08: Remove defunct "download" route in Files pointing to nonexistent controller (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

files/

application/

PhabricatorFilesApplication.php

1 line

Diff 45890

View Options

src/applications/files/application/PhabricatorFilesApplication.php

Remove defunct "download" route in Files pointing to nonexistent controllerClosedPublicActions