Maniphest T5922

File info page doesn't support public users
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	epriestley
	Aug 20 2014, 1:57 PM

Tags

Referenced Files

None

Subscribers

Description

The /file/info/asldnfasf/ page doesn't allow public users, but should.

Revisions and Commits

rP Phabricator
	D10309	rP8dd4d5cfe5e6 Files - make file info page public

Related Objects
Search...

		Status	Assigned	Task
		Resolved	chad	T4830 Applications with Public permissions requiring users to login
		Resolved	btrahan	T5922 File info page doesn't support public users

Event Timeline

epriestley created this task.Aug 20 2014, 1:57 PM

epriestley raised the priority of this task from to Normal.

epriestley updated the task description. (Show Details)

epriestley added a project: Files.

epriestley added a subscriber: epriestley.

djc added a subscriber: djc.Aug 20 2014, 2:00 PM

aklapper added a subscriber: aklapper.Aug 20 2014, 4:48 PM

The file info page should return true from shouldAllowPublic().
Same for the file list page, if it doesn't already.
Bonus points, but I'd like to make /Fxxx the authoritative file info page for consistency, and make /file/info/FILE-PHID-derpderp/ redirect to that. I think they work the opposite way right now.

btrahan added a revision: D10309: Files - make file info page public.Aug 20 2014, 8:13 PM

Closed by commit rP8dd4d5cfe5e6.