Project

General

Profile

Actions

Feature #2695

closed

Create Private File Uploads for Blogs

Added by Matt Gold over 10 years ago. Updated over 7 years ago.

Status:
Resolved
Priority name:
Normal
Assignee:
Category name:
WordPress (misc)
Target version:
Start date:
2013-08-11
Due date:
% Done:

0%

Estimated time:
Deployment actions:

Description

As noted on the wp-edu listserv and discussed via email by Matt and Boone, WordPress blogs can be made private, but file uploads are still accessible via direct link, and google apparently indexes PDFs uploaded in that way. This seems like an important oversight with consequences for user privacy. My suggestion was that there be a way for users to trigger a private file upload process during upload that stores files in a more private way (thus avoiding a solution where something is done to make all site files, including images, etc., private, which could slow down site speeds and cause needless annoyances), but Boone notes that fixing this would require an overhaul of how WP handles file uploads. Creating this ticket for future thoughts on the subject.


Related issues

Related to CUNY Academic Commons - Feature #6332: Allow uploaded files to be marked as private in an ad hoc wayNew2016-10-17

Actions
Actions #1

Updated by Matt Gold almost 10 years ago

I cannot understand how/why this hasn't been addressed in WP core yet. Isn't this a major privacy vulnerability?

Actions #2

Updated by Boone Gorges almost 10 years ago

There is no functionality in WordPress to make a blog private. We add this functionality with the More Privacy Options plugin. That's the first reason.

Limiting file access to logged-in users means diverting all downloads through the WordPress application, rather than letting the fileserver serve them directly. That can create an enormous increase in server requirements, since a single pageload might load WP dozens of times (for each image). This is a second reason.

WordPress also has to deal with a number of different server environments (various versions of Apache, Nginx, IIS, and so forth), all of which handle file protection in different and incompatible ways. On some of these setups (Nginx in particular), there simply is no way to block files in an way that's ad hoc and determined by the PHP application. All such config has to happen at the server config level, which is inaccessible to WP. This is another reason for there being no general solution for this in WP (or elsewhere). Obviously, this is not so much an issue for a site like the Commons, which runs in a specific webserver environment.

Actions #3

Updated by Boone Gorges over 8 years ago

  • Target version changed from Future release to 1.10

I have the beginnings of a tool that I built for the City Tech OpenLab. It automatically protects blog uploads based on the privacy setting of the blog where they're uploaded. It doesn't allow for per-upload or per-post privacy levels for attachments, but it's better than nothing.

Actions #4

Updated by Matt Gold over 8 years ago

Thanks, Boone. This does sound promising.

Actions #5

Updated by Boone Gorges over 7 years ago

  • Status changed from Assigned to Resolved

I've just spent some time revisiting this.

The solution on the OpenLab has two parts:
1. A plugin I wrote, called dlblock, dynamically generates an .htaccess file that disables direct access to uploaded files belonging to sites with blog_public < 0.
2. A custom file handler that replaces ms-files.php, and checks each site's blog_public before serving /files/ content.

After some investigation, it looks like 1 is not necessary on the Commons. We don't allow direct access to any of our files: the regex that we currently have in .htaccess catches any request that contains /files/, which includes foo.commons.gc.cuny.edu/files/bar.jpg and foo.commons.gc.cuny.edu/wp-content/blogs.dir/1234/files/bar.jpg. So it's enough to write the logic into our own custom file handler, which already exists for the purpose of forum attachment protection.

I've added the protection in https://github.com/cuny-academic-commons/cac/commit/fac72ac38fa0b63650ebc26e7bf36e41f7d7ff61. This feature should get a bit of testing by the community team in the lead-up to release, but I think we can marked resolved until that happens. (I'll open a separate ticket for the more complicated question of how to protect specific uploads in an ad hoc way.)

Actions #6

Updated by Matt Gold over 7 years ago

This is great, Boone -- thank you!!!

Actions #7

Updated by Boone Gorges almost 5 years ago

  • Related to Feature #6332: Allow uploaded files to be marked as private in an ad hoc way added
Actions

Also available in: Atom PDF