Skip to content

Commit c102620

Browse files
author
epriestley
committedMar 3, 2017
Lock files.video-mime-types config option for consistency
Summary: This is a consistency change to make this option consistent with `audio-mime-types`, `image-mime-types` and `icon-mime-types`, all of which are locked. (They're locked because SVG is definitely dangerous, and other types might be dangerous or might become dangerous in the future, although I'm not aware of any actual dangers from video types today.) Test Plan: Viewed `files.video-mime-types` in Config, saw it was locked. Reviewers: chad Reviewed By: chad Differential Revision: https://secure.phabricator.com/D17454
1 parent 0e7a562 commit c102620

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed
 

‎src/applications/files/config/PhabricatorFilesConfigOptions.php

+1
Original file line numberDiff line numberDiff line change
@@ -155,6 +155,7 @@ public function getOptions() {
155155
'List of MIME types which can be rendered with an `%s` tag.',
156156
'<audio />')),
157157
$this->newOption('files.video-mime-types', 'set', $video_default)
158+
->setLocked(true)
158159
->setSummary(pht('Configure which MIME types are video.'))
159160
->setDescription(
160161
pht(

0 commit comments

Comments
 (0)
Failed to load comments.