Page MenuHomePhabricator

Default upload visibility should be "Public (No Login Required)"
Closed, ResolvedPublic

Description

Upstream: https://secure.phabricator.com/T6564

The default visibility for files created at https://phabricator.wikimedia.org/file/upload/ should be "Public (No Login Required)". This is the same behavior as Bugzilla. Currently, the default is "All Users" which requires a login.

If possible, creating non-public files (or changing visibility of an existing file to non-public) should be limited to the Security group,

Event Timeline

Mattflaschen-WMF renamed this task from Default upload visibility should be Public (No Login Required) to Default upload visibility should be "Public (No Login Required)".
Mattflaschen-WMF raised the priority of this task from to High.
Mattflaschen-WMF updated the task description. (Show Details)
Mattflaschen-WMF changed Security from none to None.
Mattflaschen-WMF added a subscriber: Qgil.

/me wondering why this is set to high priority?

(or changing visibility of an existing file to non-public)

For the records, https://www.mediawiki.org/wiki/Phabricator/Help#Uploading_file_attachments states:

Note that you cannot upload a file and then decrease the access level to the file (e.g linking it to a restricted security ticket). You would have to delete the file and reupload it with stricter access permissions. It is recommended to upload files which should have restricted access together with the creation of a restricted (security) ticket.

Qgil lowered the priority of this task from High to Medium.EditedNov 12 2014, 8:33 AM

First I wanted to check whether the files created via drag&drop are public, and they are. Then I checked the files at bugzillapreview, and they are also public. Therefore, this problem affects exclusively files uploaded via https://phabricator.wikimedia.org/file/upload/. Still a problem, but not high priority.

I could not find in phab-01 any configuration to set the default to Public. It would make sense for new uploads to have the same default Can View policy as "Can Use Application" (which is set to Public).

I couldn't find either a policy to limit who can set the Can View policy of files uploaded (just like Maniphest has one, and this is why regular users only have the option of public/private.

@Mattflaschen , since you have experience reporting problems upstream, would you like to report these two (separate or together, your choice).

/me wondering why this is set to high priority?

Because it's important that users (simply using the default setting without thinking about it) don't create a huge mess of incorrectly-permissioned files. However, it's true that on further review it's not that bad since most people probably won't even know about the upload file page (the drag and drop message when you click the upload icon does not currently mention it).

Note that you cannot upload a file and then decrease the access level to the file (e.g linking it to a restricted security ticket). You would have to delete the file and reupload it with stricter access permissions. It is recommended to upload files which should have restricted access together with the creation of a restricted (security) ticket.

Thanks.

In T1248#21580, @Qgil wrote:

I could not find in phab-01 any configuration to set the default to Public. It would make sense for new uploads to have the same default Can View policy as "Can Use Application" (which is set to Public).

Filed as https://secure.phabricator.com/T6564 ("Set default permissions for file uploads to same as File application")

I couldn't find either a policy to limit who can set the Can View policy of files uploaded (just like Maniphest has one, and this is why regular users only have the option of public/private.

Filed as https://secure.phabricator.com/T6565 ("Allow limiting which "Can View" settings can be used for files").

Qgil lowered the priority of this task from Medium to Low.Nov 27 2014, 8:17 AM

This is Low priority upstream.

Upstream has resolved its part (creating the policy). https://secure.phabricator.com/applications/view/PhabricatorFilesApplication/ now specifies

Default View Policy: All Users

After upgrading Wikimedia Phabricator, we need to set this policy as "Public (No Login Required".

Qgil changed the task status from Open to Stalled.Nov 30 2014, 8:07 PM
Qgil edited projects, added Phabricator; removed Phabricator (Upstream).
Aklapper claimed this task.

This is the case now.

https://phabricator.wikimedia.org/applications/view/PhabricatorFilesApplication/ states

Default View Policy: Public (No Login Required)

and going to https://phabricator.wikimedia.org/file/upload/ shows "Public" as default.