Skip to content

Commit b14c01d

Browse files
committed
Whitelist mutable manifest keys
1 parent 1f27c0c commit b14c01d

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

app/controllers/videos_controller.rb

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,11 @@ def update_video_with_manifest(video, manifest)
166166
return :ok
167167
end
168168

169+
mutable_keys = ["title", "genre", "tag", "formatVersion", "annotations"] + ignore
170+
if @video.read_manifest.except(*mutable_keys) != manifest.except(*mutable_keys)
171+
return :forbidden
172+
end
173+
169174
parent_rev = manifest["revision"]
170175
if parent_rev != @video.revision_num
171176
parent_manifest = @video.manifest_revision(parent_rev)

0 commit comments

Comments
 (0)