fix CSP / X-Frame-Options for media embeds (mastodon#9558)

ProjectMyosotis · Oct 2, 2019 · e94794d · e94794d
1 parent 69faeb2
commit e94794d
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/app/controllers/media_controller.rb b/app/controllers/media_controller.rb
@@ -6,12 +6,17 @@ class MediaController < ApplicationController
   before_action :set_media_attachment
   before_action :verify_permitted_status!
 
+  content_security_policy only: :player do |p|
+    p.frame_ancestors(false)
+  end
+
   def show
     redirect_to @media_attachment.file.url(:original)
   end
 
   def player
     @body_classes = 'player'
+    response.headers['X-Frame-Options'] = 'ALLOWALL'
     raise ActiveRecord::RecordNotFound unless @media_attachment.video? || @media_attachment.gifv?
   end