Page MenuHomePhabricator

Upload a new version bypasses file extension checks
Closed, ResolvedPublic

Description

When you choose "upload a new version", you can upload an arbitrary file type. For instance, you can upload a gif and then reupload a BMP file on top of it.

By default this will succeed, because by default the "ignore warnings" option is checked. This cannot be the intent. The ignore warnings option should probably not be checked by default.


Version: unspecified
Severity: major

Details

Reference
bz22933

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 11:00 PM
bzimport set Reference to bz22933.
bzimport added a subscriber: Unknown Object (MLST).

The upload form itself only gives the warning "There's already a file with that name. Do you want to overwrite it?", and it does not say anything about the changed filetype either.

This issue is fixed in trunk, but still broken in wmf-deployment.

Bryan.TongMinh wrote:

Probably fixed with the recent deployment?